cvs: gd /libgd/src gdft.c

tabe Wed Apr 15 17:24:45 2009 UTC Modified files: /gd/libgd/src gdft.c Log: decrement the pattern reference count in case of failure. http://cvs.php.net/viewvc.cgi/gd/libgd/src/gdft.c?r1=1.40&r2=1.41&diff_format=u Index: gd/libgd/src/gdft.c diff -u gd/libgd/src/gdft.c:1.40 gd/libgd/src/gdft.c:1.41 --- gd/libgd/src/gdft.c:1.40 Wed Apr 15 16:19:15 2009 +++ gd/libgd/src/gdft.c Wed Apr 15 17:24:45 2009 @@ -1517,8 +1517,12 @@ font = find_font(pattern); FcPatternDestroy(pattern); - if (!font || FcPatternGetString(font, FC_FAMILY, 0, &family) != FcResultMatch) - return "fontconfig: Couldn't retrieve font family name."; + if (!font) + return "fontconfig: Couldn't find font."; + if (FcPatternGetString(font, FC_FAMILY, 0, &family) != FcResultMatch) { + FcPatternDestroy(font); + return "fontconfig: Couldn't retrieve font family name."; + } /* Check whether we got the font family we wanted. */ if (strcmp((const char *)family, postscript_alias[i].family) != 0) { @@ -1558,8 +1562,12 @@ } #endif - if (!font || FcPatternGetString(font, FC_FILE, 0, &file) != FcResultMatch) + if (!font) + return "fontconfig: Couldn't find font."; + if (FcPatternGetString(font, FC_FILE, 0, &file) != FcResultMatch) { + FcPatternDestroy(font); return "fontconfig: Couldn't retrieve font file name."; + } *fontpath = strdup((const char *)file);