Skip to content

Commit

Permalink
Merge pull request opencv#14664 from alalek:oss_fuzz_13314
Browse files Browse the repository at this point in the history
  • Loading branch information
alalek committed May 29, 2019
2 parents 216988b + 419a6d5 commit d2d8535
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 4 deletions.
17 changes: 17 additions & 0 deletions 3rdparty/libpng/patches/20190528-fix-leak-png_handle_exif.diff
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
diff --git a/3rdparty/libpng/pngrutil.c b/3rdparty/libpng/pngrutil.c
index d5fa08c397..4db3de990b 100644
--- a/3rdparty/libpng/pngrutil.c
+++ b/3rdparty/libpng/pngrutil.c
@@ -2087,10 +2087,8 @@ png_handle_eXIf(png_structrp png_ptr, png_inforp info_ptr, png_uint_32 length)
}
}

- if (png_crc_finish(png_ptr, 0) != 0)
- return;
-
- png_set_eXIf_1(png_ptr, info_ptr, length, info_ptr->eXIf_buf);
+ if (png_crc_finish(png_ptr, 0) == 0)
+ png_set_eXIf_1(png_ptr, info_ptr, length, info_ptr->eXIf_buf);

png_free(png_ptr, info_ptr->eXIf_buf);
info_ptr->eXIf_buf = NULL;
6 changes: 2 additions & 4 deletions 3rdparty/libpng/pngrutil.c
Original file line number Diff line number Diff line change
Expand Up @@ -2087,10 +2087,8 @@ png_handle_eXIf(png_structrp png_ptr, png_inforp info_ptr, png_uint_32 length)
}
}

if (png_crc_finish(png_ptr, 0) != 0)
return;

png_set_eXIf_1(png_ptr, info_ptr, length, info_ptr->eXIf_buf);
if (png_crc_finish(png_ptr, 0) == 0)
png_set_eXIf_1(png_ptr, info_ptr, length, info_ptr->eXIf_buf);

png_free(png_ptr, info_ptr->eXIf_buf);
info_ptr->eXIf_buf = NULL;
Expand Down

0 comments on commit d2d8535

Please sign in to comment.