Merge pull request #7923 from Carreau/free-all-the-fonts

NelleV · web-flow · commit c93142f3d906 · 2017-01-22T17:08:04.000-08:00
Fix leak of filedescriptor if fontsize cannot be set.
diff --git a/src/ft2font.cpp b/src/ft2font.cpp
@@ -508,6 +508,7 @@ FT2Font::FT2Font(FT_Open_Args &open_args, long hinting_factor_) : image(), face(
 
     error = FT_Set_Char_Size(face, 12 * 64, 0, 72 * (unsigned int)hinting_factor, 72);
     if (error) {
+        FT_Done_Face(face);
         throw "Could not set the fontsize";
     }
 

Original file line number	Diff line number	Diff line change
`@@ -508,6 +508,7 @@ FT2Font::FT2Font(FT_Open_Args &open_args, long hinting_factor_) : image(), face(`
`508`	`508`
`509`	`509`	`error = FT_Set_Char_Size(face, 12 * 64, 0, 72 * (unsigned int)hinting_factor, 72);`
`510`	`510`	`if (error) {`
	`511`	`+ FT_Done_Face(face);`
`511`	`512`	`throw "Could not set the fontsize";`
`512`	`513`	`}`
`513`	`514`