[cairo-bugs] [Bug 105466] Memory leak in cairo_show_text

[bugzilla-daemon at freedesktop.org]

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

--- Comment #8 from Philip Chimento <philip.chimento at gmail.com> ---
(In reply to Uli Schlachter from comment #7)
> Call FcFini() to shutdown fontconfig.

Would you consider a patch to call this in cairo_debug_reset_static_data()?
It's probably fine as it is, but on the other hand it's a
not-easily-discoverable burden to callers if they have to link directly to all
of Cairo's dependent libraries to call their respective deinitialization
functions. (I can't think of a use case when you _wouldn't_ want to deinit
fontconfig while deiniting Cairo, but maybe there is one?)

> And if you start seeing a 256k memory leak from pixman_glyph_cache_create(),
> update to a cairo version that includes 1ed124ace20194.
> 
> With the above changes, here is my valgrind output for the example from
> comment #5:
> 
> [snip]
>
> (No, pixman does not provide a function to destroy its global_implementation
> variable that is allocated by _pixman_choose_implementation(); no, this is
> not a real memory leak; no, this is not cairo's "fault")

Yes, I know the difference between a memory leak and static data that is held
until program shutdown. I am not accusing anybody or saying that anything is
anybody's fault.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.cairographics.org/archives/cairo-bugs/attachments/20180409/911b8f8f/attachment.html>