diff options
| author | Andy Maloney <asmaloney@gmail.com> | 2013-02-10 08:40:52 -0500 |
|---|---|---|
| committer | S. Davilla <davilla@4pi.com> | 2013-02-19 11:35:35 -0500 |
| commit | 9f0caa3cb87404746d00fab1d9974af4a7ff2ef4 (patch) | |
| tree | a26281c1ea340a5699a5b510ee0998dc8d084c34 | |
| parent | 3922bf106ee4af5a4f69953be14d272de62c0610 (diff) | |
Close file properly on error
| -rw-r--r-- | xbmc/visualizations/Goom/goom2k4-0/src/pngload.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/xbmc/visualizations/Goom/goom2k4-0/src/pngload.c b/xbmc/visualizations/Goom/goom2k4-0/src/pngload.c index 98ed297fa7..5335eb2772 100644 --- a/xbmc/visualizations/Goom/goom2k4-0/src/pngload.c +++ b/xbmc/visualizations/Goom/goom2k4-0/src/pngload.c @@ -30,6 +30,7 @@ loadpng (char *file_name, int *w, int *h, unsigned int ***buf) png_ptr = png_create_read_struct (PNG_LIBPNG_VER_STRING, (png_voidp) NULL, NULL, NULL); if (!png_ptr) { + fclose (fp); fprintf (stderr, "Memory error\n"); return 1; } @@ -37,6 +38,7 @@ loadpng (char *file_name, int *w, int *h, unsigned int ***buf) info_ptr = png_create_info_struct (png_ptr); if (!info_ptr) { png_destroy_read_struct (&png_ptr, (png_infopp) NULL, (png_infopp) NULL); + fclose (fp); fprintf (stderr, "Read error 1\n"); return 1; } @@ -44,6 +46,7 @@ loadpng (char *file_name, int *w, int *h, unsigned int ***buf) end_info = png_create_info_struct (png_ptr); if (!end_info) { png_destroy_read_struct (&png_ptr, &info_ptr, (png_infopp) NULL); + fclose (fp); fprintf (stderr, "Read error 2\n"); return 1; } |