[fontconfig] Fix memory leaks in FcFreeTypeQueryFace().
Akira TAGOH
tagoh at fedoraproject.org
Fri Sep 13 08:35:30 UTC 2013
commit a49dc69446ef0adbab127e5fe935b0aa382d98b0
Author: Akira TAGOH <tagoh at redhat.com>
Date: Fri Sep 13 17:35:18 2013 +0900
Fix memory leaks in FcFreeTypeQueryFace().
fontconfig-fix-memleak.patch | 69 ++++++++++++++++++++++++++++++++++++++++++
fontconfig.spec | 7 ++++-
2 files changed, 75 insertions(+), 1 deletions(-)
---
diff --git a/fontconfig-fix-memleak.patch b/fontconfig-fix-memleak.patch
new file mode 100644
index 0000000..f7753ea
--- /dev/null
+++ b/fontconfig-fix-memleak.patch
@@ -0,0 +1,69 @@
+From a61e145304da86c8c35b137493bbd8fd5dd1e7f5 Mon Sep 17 00:00:00 2001
+From: Akira TAGOH <akira at tagoh.org>
+Date: Mon, 9 Sep 2013 19:59:31 +0900
+Subject: [PATCH] Fix memory leaks in FcFreeTypeQueryFace
+
+---
+ src/fcfreetype.c | 15 ++++++++++-----
+ 1 file changed, 10 insertions(+), 5 deletions(-)
+
+diff --git a/src/fcfreetype.c b/src/fcfreetype.c
+index 02e85cb..e394307 100644
+--- a/src/fcfreetype.c
++++ b/src/fcfreetype.c
+@@ -1105,7 +1105,6 @@ FcFreeTypeQueryFace (const FT_Face face,
+ const char *tmp;
+
+ FcChar8 *hashstr = NULL;
+- char *fontdata = NULL;
+ FT_Error err;
+ FT_ULong len = 0, alen;
+
+@@ -1668,15 +1667,21 @@ FcFreeTypeQueryFace (const FT_Face face,
+ err = FT_Load_Sfnt_Table (face, 0, 0, NULL, &len);
+ if (err == FT_Err_Ok)
+ {
++ char *fontdata;
++
+ alen = (len + 63) & ~63;
+ fontdata = malloc (alen);
+ if (!fontdata)
+ goto bail3;
+ err = FT_Load_Sfnt_Table (face, 0, 0, (FT_Byte *)fontdata, &len);
+ if (err != FT_Err_Ok)
++ {
++ free (fontdata);
+ goto bail3;
++ }
+ memset (&fontdata[len], 0, alen - len);
+ hashstr = FcHashGetSHA256DigestFromMemory (fontdata, len);
++ free (fontdata);
+ }
+ else if (err == FT_Err_Invalid_Face_Handle)
+ {
+@@ -1692,7 +1697,11 @@ FcFreeTypeQueryFace (const FT_Face face,
+ if (hashstr)
+ {
+ if (!FcPatternAddString (pat, FC_HASH, hashstr))
++ {
++ free (hashstr);
+ goto bail1;
++ }
++ free (hashstr);
+ }
+ bail3:
+
+@@ -1783,10 +1792,6 @@ bail3:
+ bail2:
+ FcCharSetDestroy (cs);
+ bail1:
+- if (hashstr)
+- free (hashstr);
+- if (fontdata)
+- free (fontdata);
+ FcPatternDestroy (pat);
+ bail0:
+ return NULL;
+--
+1.8.3.1
+
diff --git a/fontconfig.spec b/fontconfig.spec
index fe99682..11857d3 100644
--- a/fontconfig.spec
+++ b/fontconfig.spec
@@ -3,7 +3,7 @@
Summary: Font configuration and customization library
Name: fontconfig
Version: 2.10.95
-Release: 3%{?dist}
+Release: 4%{?dist}
# src/ftglue.[ch] is in Public Domain
# src/fccache.c contains Public Domain code
# fc-case/CaseFolding.txt is in the UCD
@@ -17,6 +17,7 @@ Source1: 25-no-bitmap-fedora.conf
# https://bugzilla.redhat.com/show_bug.cgi?id=140335
Patch0: fontconfig-2.8.0-sleep-less.patch
Patch1: fontconfig-no-dir-when-no-conf.patch
+Patch2: fontconfig-fix-memleak.patch
BuildRequires: expat-devel
BuildRequires: freetype-devel >= %{freetype_version}
@@ -59,6 +60,7 @@ which is useful for developing applications that uses fontconfig.
%setup -q
%patch0 -p1 -b .sleep-less
%patch1 -p1 -b .nodir
+%patch2 -p1 -b .memleak
%build
# We don't want to rebuild the docs, but we want to install the included ones.
@@ -135,6 +137,9 @@ fi
%doc fontconfig-devel.txt fontconfig-devel
%changelog
+* Fri Sep 13 2013 Akira TAGOH <tagoh at redhat.com> - 2.10.95-4
+- Fix memory leaks in FcFreeTypeQueryFace().
+
* Mon Sep 2 2013 Akira TAGOH <tagoh at redhat.com> - 2.10.95-3
- Do not create a directory for migration when no old config file and directory.
(#1003495)
More information about the fonts-bugs
mailing list