[Fontconfig] Patch: fix memleak (spotted by Coverity)
Frederic Crozat
fcrozat at mandriva.com
Mon Apr 10 09:31:01 PDT 2006
Le lundi 10 avril 2006 à 12:07 -0400, Patrick Lam a écrit :
> Frederic Crozat wrote:
> > The following patch fixes memory and file descriptor leaks (Coverity
> > reports #777 and #1826) in error cases for FcDirScanConfig.
> >
> > Please review this patch carefully, as I used goto for bail conditions
> > and tried to call free functions only one time in the FcDirScanConfig
> > code (to ease code review in the future).
>
> I've committed a modified version of this patch. I think it's correct.
It looks ok to me.
To celebrate, here is another memleak fix (Coverity report #1824) for
FcDirCacheUnlink ;)
Don't worry, there are only 28 defects left to check, I'll probably
continue tomorrow...
--
Frederic Crozat <fcrozat at mandriva.com>
Mandriva
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fontconfig-2.3.94-hashmemleak2.patch
Type: text/x-patch
Size: 1129 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/fontconfig/attachments/20060410/842963e6/fontconfig-2.3.94-hashmemleak2.bin
More information about the Fontconfig
mailing list