Page 1 of 1

[patch_done] Memory leak on TZIBEventAlerter

Posted: 20.08.2008, 14:39
by magnomp
After installing Zeos 6.6.3 and using it's TZIBEventAlerter I've noticed some memory leaking on my application.
This component instantiates a TStringList (FEvents) on it's constructor but never frees it.
I've added a call to FEvents.Free on TZIBEventAlerter destructor. Further, I've removed the following lines:
if Assigned(FEvents) then
TStringList(FEvents).OnChange := nil;
Because if FEvents is gonna be freed, there's no point on setting nil to OnChange.

Posted: 20.08.2008, 20:34
by mdaems
Committed to testing branch. I'll port the change to 6.6-patches branch in about a week. SVN Rev. 425.

Mark

Posted: 24.08.2008, 18:44
by mdaems
Now available in the 6.6-patches branch.