shanapu / MyJailbreak

A rewrite of Franugs Special Jailbreak and merge of eccas, ESK0s & zipcores Jailbreak Warden
https://forums.alliedmods.net/showthread.php?t=283212
GNU General Public License v3.0
51 stars 36 forks source link

Memory leak #330

Open shanapu opened 5 years ago

shanapu commented 5 years ago

Creating Handles without deleting. Maybe also on other spots... make own tests with sm_dump_handles

https://forums.alliedmods.net/showpost.php?p=2633463&postcount=27

Franc1sco commented 5 years ago

https://github.com/shanapu/MyJailbreak/blob/dev/addons/sourcemod/scripting/MyJailbreak/weapons.sp#L296-L299 this need to be closed first before of create more and more menus.

nanociaglia commented 5 years ago

screenshot_1

weapons.smx is in the top5 plugins with highest memory size

Franc1sco commented 5 years ago

I fixed it here https://github.com/shanapu/MyJailbreak/pull/333

Hexer10 commented 5 years ago

After more than ~50 map changes: image

Seems like we should look into it, in weapons, warden and myjb core

Thanks for Bara for his help here.

EDIT 1: If we remove all the event days, the myjb leaks will "stop" so there are issues with the API, that is being called by the event days every map change.

Hexer10 commented 5 years ago

This should be fixed with https://github.com/shanapu/MyJailbreak/commit/b2185cee9c84c689d0b1c148aae16855f452dfd7 . Please reply here if you're having further issues.

nanociaglia commented 5 years ago

that's on dev branch

Hexer10 commented 5 years ago

I know, but you should be able to use it without any major issues

shanapu commented 5 years ago

Thank you all guys! I have very little time lately to take care of my plugins. It's great to see you working together and solving the problem yourself.

Thank you very very much for your support!

Sharing is caring! ❤️