[Poppler-bugs] [Bug 89952] Memory Leak in CairoFreeTypeFont::create

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=89952

Carlos Garcia Campos <carlosgc at gnome.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #8 from Carlos Garcia Campos <carlosgc at gnome.org> ---
(In reply to Jason Crain from comment #7)
> (In reply to Carlos Garcia Campos from comment #6)
> > You should also free the bytes in _ft_new_face when FT_New_Memory_Face
> > fails, no?
> 
> It should be fine because when FT_New_Memory_Face fails, _ft_new_face will
> return false, and CairoFreeTypeFont::create will free it in the err2 label.

Ah, I missed the gfree(font_data) in err2: label part. Pushed to git master
now, thanks!

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/poppler-bugs/attachments/20150413/adfd6e3c/attachment.html>