Skip to content

Draft: fix memleak. (issue #748)

Ziggy requested to merge Ziggy/pango:main into main

Adding a missing FcPatternDestroy (fontset->fonts[i]) to pangofc-fontmap.c#1735.

Although there is FcFontSetDestroy (fontset) in pangofc-fontmap.c#1737 which in turn calls FcPatternDestroy - for some reason the array (size?) changes and thus isn't being destroyed and results with a memleak. To try it yourself, run tests/test-font via valgrind and watch the leak. Once you add the FcPatternDestroy (fontset->fonts[i]), all is well.

Merge request reports