php-groups.com | Post Question | Search | About | Groups | Contact | Register | Login


New in This Group

cvs: gd /gd-pango CMakeLists.txt gd_pango.c /gd-pango/cmake/modules FindGD.cmake /gd-pango/examples CMakeLists.txt /gd-pango/tests CMakeLists.txt /gd-pango/tests/pango CMakeLists.txt
8/31/2008 5:07:09 PM

cvs: gd /libgd/src gd_crop.c
8/31/2008 4:57:18 PM

cvs: gd /libgd/src gdxpm.c /libgd/tests/xpm CMakeLists.txt Makefile.am color_name.c color_name.xpm
8/21/2008 11:34:05 AM

cvs: gd /libgd/src gd_color_map.c
8/21/2008 11:10:53 AM

cvs: gd /libgd/src CMakeLists.txt
8/15/2008 3:08:55 AM

cvs: gd(GD_2_0) /libgd/src gdxpm.c /libgd/src/tests CMakeLists.txt Makefile.am /libgd/src/tests/xpm CMakeLists.txt Makefile.am bug00166.c bug00166.xpm
8/14/2008 10:48:29 AM

cvs: gd /libgd/tests/xpm CMakeLists.txt Makefile.am bug00166.c bug00166.xpm
8/14/2008 10:45:42 AM

cvs: gd /libgd/src gdxpm.c /libgd/tests CMakeLists.txt Makefile.am
8/14/2008 10:41:57 AM

cvs: gd(GD_2_0) /libgd/src gdfx.c index.html
8/6/2008 7:16:44 AM

cvs: gd /libgd/src gdfx.c
8/6/2008 7:16:28 AM

cvs: gd(GD_2_0) /libgd/src gd_png.c Reply

Group: php.gd.cvs Subscribe Posted:3/14/2009 4:17:14 AM Replies:0 Views:31 Items(0)
tabe Sat Mar 14 04:17:14 2009 UTC

Modified files: (Branch: GD_2_0)
/gd/libgd/src gd_png.c
Log:
MFH: fixed a slim chance of memory leaks.

http://cvs.php.net/viewvc.cgi/gd/libgd/src/gd_png.c?r1=1.21.2.5&r2=1.21.2.6&diff_format=u
Index: gd/libgd/src/gd_png.c
diff -u gd/libgd/src/gd_png.c:1.21.2.5 gd/libgd/src/gd_png.c:1.21.2.6
--- gd/libgd/src/gd_png.c:1.21.2.5 Fri Mar 13 20:26:55 2009
+++ gd/libgd/src/gd_png.c Sat Mar 14 04:17:14 2009
@@ -1,4 +1,4 @@
-/* $Id: gd_png.c,v 1.21.2.5 2009/03/13 20:26:55 tabe Exp $ */
+/* $Id: gd_png.c,v 1.21.2.6 2009/03/14 04:17:14 tabe Exp $ */
#ifdef HAVE_CONFIG_H
#include "config.h"
#endif
@@ -743,14 +743,13 @@
png_bytep *prow_pointers;
int saveAlphaFlag = im->saveAlphaFlag;
if (overflow2(sizeof (png_bytep), height)) {
- return;
+ goto bail;
}
row_pointers = gdMalloc (sizeof (png_bytep) * height);
if (row_pointers == NULL)
{
fprintf (stderr, "gd-png error: unable to allocate row_pointers\n");
- /* 2.0.29: return was missing */
- return;
+ goto bail;
}
prow_pointers = row_pointers;
for (j = 0; j < height; ++j)
@@ -763,7 +762,7 @@
gdFree (row_pointers[i]);
/* 2.0.29: memory leak TBB */
gdFree(row_pointers);
- return;
+ goto bail;
}
pOutputRow = *prow_pointers++;
pThisRow = *ptpixels++;
@@ -801,15 +800,13 @@
{
png_bytep *row_pointers;
if (overflow2(sizeof (png_bytep), height)) {
- return;
+ goto bail;
}
row_pointers = gdMalloc (sizeof (png_bytep) * height);
if (row_pointers == NULL)
{
- fprintf (stderr,
- "gd-png error: unable to allocate row_pointers\n");
- /* TBB: return missing */
- return;
+ fprintf (stderr, "gd-png error: unable to allocate row_pointers\n");
+ goto bail;
}
for (j = 0; j < height; ++j)
{
@@ -820,7 +817,7 @@
gdFree (row_pointers[i]);
/* TBB: memory leak */
gdFree (row_pointers);
- return;
+ goto bail;
}
for (i = 0; i < width; ++i)
row_pointers[j][i] = mapping[im->pixels[j][i]];


t...@php.net ("Takeshi Abe")
3/14/2009 4:17:14 AM





Reply:
| or Email(not shown): Name: | Register | Login

Alert me when someone responds to this posting.











Newest Articles

note 97095 added to soapclient.soapclient
4/1/2010 5:58:19 AM

note 97094 added to function.sqlite-changes
4/1/2010 5:06:46 AM

note 97093 added to function.sqlite-changes
4/1/2010 5:06:21 AM

note 97092 added to ref.xmlrpc
4/1/2010 4:52:18 AM

note 97091 added to function.var-export
4/1/2010 3:32:03 AM