MantisBT - Zandronum
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0003018||Zandronum||[All Projects] Bug||public||2017-02-19 01:03||2019-06-04 21:54|
|Priority||normal||Severity||major||Reproducibility||have not tried|
|Target Version||Fixed in Version|
|Summary||0003018: Master server leaks memory|
|Description||NETBUFFER_s does not destroy its buffer when deleted, but relies on its Free method being called. Master server's BanlistPacketSender does not do this. As a result, 1KiB of memory is leaked every time the master server sends the banlist to a server. Since there's hundreds of servers, this must tally up to hundreds of kilobytes lost quickly enough.|
Note: I don't know how to actually test it. But this seems like a glaring problem and I can't see how this cannot be happening.
The master server as is locks up mysteriously and needs to be restarted all the time. Maybe this is why?
I noticed this by doing some refactor on the network classes and decided to make NETBUFFER_s have a destructor. I noticed that this makes every single Free() calls unnecessary and allows removal of a lot of code.
|Steps To Reproduce|
|Tags||No tags attached.|
|2017-02-19 01:03||Dusk||New Issue|
|2017-02-19 01:03||Dusk||Status||new => assigned|
|2017-02-19 01:03||Dusk||Assigned To||=> Dusk|
|2017-02-19 01:07||Dusk||Note Added: 0016877|
|2019-06-04 21:54||Dusk||Status||assigned => new|