shithub: freetype+ttf2subf

Download patch

ref: fe0a7d9df5b45430b6a36627c857f0393c77fbb5
parent: 134de096e07c4b700ccff5226fcfe4c90ae2ed04
author: Werner Lemberg <[email protected]>
date: Thu Jul 13 06:25:42 EDT 2017 

[base] Fix memory leak.

Reported as

  https://bugs.chromium.org/p/chromium/issues/detail?id=738362

* src/base/ftglyph.c (FT_Get_Glyph): Do proper deallocation in case
of error.

git/fs: mount .git/fs: mount/attach disallowed 
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2017-07-13  Werner Lemberg  <[email protected]>
+
+	[base] Fix memory leak.
+
+	Reported as
+
+	  https://bugs.chromium.org/p/chromium/issues/detail?id=738362
+
+	* src/base/ftglyph.c (FT_Get_Glyph): Do proper deallocation in case
+	of error.
+
 2017-07-12  Werner Lemberg  <[email protected]>
 
 	[base] Integer overflow.
--- a/src/base/ftglyph.c
+++ b/src/base/ftglyph.c
@@ -412,13 +412,15 @@
          slot->advance.x <= -0x8000L * 64 )
     {
       FT_ERROR(( "FT_Get_Glyph: advance width too large\n" ));
-      return FT_THROW( Invalid_Argument );
+      error = FT_THROW( Invalid_Argument );
+      goto Exit2;
     }
     if ( slot->advance.y >=  0x8000L * 64 ||
          slot->advance.y <= -0x8000L * 64 )
     {
       FT_ERROR(( "FT_Get_Glyph: advance height too large\n" ));
-      return FT_THROW( Invalid_Argument );
+      error = FT_THROW( Invalid_Argument );
+      goto Exit2;
     }
 
     glyph->advance.x = slot->advance.x * 1024;
@@ -427,6 +429,7 @@
     /* now import the image from the glyph slot */
     error = clazz->glyph_init( glyph, slot );
 
+  Exit2:
     /* if an error occurred, destroy the glyph */
     if ( error )
       FT_Done_Glyph( glyph );