[freespace2] memory leak patch is in
Taylor Richards
mtrs at bellsouth.net
Mon Jun 16 08:05:41 EDT 2003
On Sat, 2003-06-14 at 20:14, Matt wrote:
> May I suggest that in the font memory leak fix, after you free the given
> resource, then set the resource to be NULL. It's a good habit to get into
> in case there are double frees or attempts to access to resource after a
> free. Also, if there are ever any duplicat entries in the list, this would
> help alert you to that situation.
Yeah I missed that, thanks for pointing it out. I know that I had some
problems get the fonts to close out without crashing but since that was
about 3-4 months ago I don't remember if there was a reason those still
aren't reset to NULL. I probably just didn't think to do it. Some
tweaks were needed in the order in which things are closed out to avoid
some crashes which was probably real the cause to begin with since those
tweaks happened later. I'll test just to make sure then go through and
try and find any others that aren't set to NULL and make the changes.
Taylor
--
Taylor Richards <mtrs at bellsouth.net>
More information about the freespace2
mailing list