AlliedModders

AlliedModders (https://forums.alliedmods.net/index.php)
-   TF2Items (https://forums.alliedmods.net/forumdisplay.php?f=146)
-   -   Memory Leak (https://forums.alliedmods.net/showthread.php?t=223414)

4bdul 08-13-2013 08:58

Memory Leak
 
Recovered from the SourceMod fatal error logs, I get this at-least once per day.

TF2Items_Manager Version: 065f4072e79a

Quote:

L 08/12/2013 - 15:54:09: [SM] MEMORY LEAK DETECTED IN PLUGIN (file "tf2items_manager.smx")
L 08/12/2013 - 15:54:09: [SM] Unloading plugin to free 6507 handles.
L 08/12/2013 - 15:54:09: [SM] Contact the author(s) of this plugin to correct this error.
L 08/12/2013 - 23:18:26: [SM] MEMORY LEAK DETECTED IN PLUGIN (file "tf2itemsinfo.smx")
L 08/12/2013 - 23:18:26: [SM] Unloading plugin to free 4902 handles.
L 08/12/2013 - 23:18:26: [SM] Contact the author(s) of this plugin to correct this error.

asherkin 08-13-2013 09:53

Re: Memory Leak
 
Those counts are very low, please post your tf2items_manager config file.

friagram 09-11-2013 11:06

Re: Memory Leak
 
Ongivenameditem doesnt close the handle automatically
Seems a lot of plugins do not close it either, you can just use a global variable, and use only this for all item creatin, and keep closing the handle every time you enter ogni... This may be your problem

Powerlord 09-11-2013 12:17

Re: Memory Leak
 
Quote:

Originally Posted by friagram (Post 2032709)
Ongivenameditem doesnt close the handle automatically
Seems a lot of plugins do not close it either, you can just use a global variable, and use only this for all item creatin, and keep closing the handle every time you enter ogni... This may be your problem

Global or static variable... the only difference is that static isn't visible outside the function (or file if global) you declare it in.

Anyway, the static trick is what I do. I was hoping we'd eventually get a new flag for closing handles after it finishes processing.


All times are GMT -4. The time now is 21:48.

Powered by vBulletin®
Copyright ©2000 - 2024, vBulletin Solutions, Inc.