Anonymous avatar Anonymous committed 893447a

* avoid memory leak in case of gdRealloc's failure
* strdup() should be accompanied by free()

Comments (0)

Files changed (1)

       /* 2.0.22: Thorben Kundinger: +8 is needed, not +6. */
       fullname = gdRealloc (fullname,
                           strlen (fontsearchpath) + strlen (name) + 8);
+      if (!fullname) {
+          free (path);
+          free (fontlist);
+          return "could not alloc full path of font";
+      }
       /* if name is an absolute or relative pathname then test directly */
 #ifdef NETWARE
       /* netware uses the format "volume:/path" or the standard "/path" */
 	    {
 	      font_found++;
               /* 2.0.16: memory leak fixed, Gustavo Scotti */
-              gdFree (path);
+              free (path);
 	      break;
 	    }
 	}
 	      break;
 	    }
 	}
-      gdFree (path);
+      free (path);
       if (font_found)
 	break;
     }
-  gdFree (fontlist);
+  free (fontlist);
   if (!font_found)
     {
       gdFree (fullname);
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.