com php-src: Fix bug #68113 (Heap corruption in exif_thumbnai l()): ext/exif/exif.c ext/exif/tests/bug681 13.jpg ext/exif/tests/bug68113.phpt

Commit: f9ba0a157f2d7e6d027285cb2ef964a919e67b8e Author: Stanislav Malyshev <stas@php.net> Sun, 28 Sep 2014 16:57:42 -0700 Parents: 71b63fc701b16e326b6ee01369d746aed2c7643c Branches: PHP-5.6 Link: http://git.php.net/?p=php-src.git;a=commitdiff;h=f9ba0a157f2d7e6d027285cb2ef964a919e67b8e Log: Fix bug #68113 (Heap corruption in exif_thumbnail()) Bugs: https://bugs.php.net/68113 Changed paths: M ext/exif/exif.c A ext/exif/tests/bug68113.jpg A ext/exif/tests/bug68113.phpt Diff: diff --git a/ext/exif/exif.c b/ext/exif/exif.c index 0b28f1c..ab2a504 100644 --- a/ext/exif/exif.c +++ b/ext/exif/exif.c @@ -2416,11 +2416,11 @@ static void* exif_ifd_make_value(image_info_data *info_data, int motorola_intel data_ptr += 8; break; case TAG_FMT_SINGLE: - memmove(data_ptr, &info_data->value.f, byte_count); + memmove(data_ptr, &info_value->f, 4); data_ptr += 4; break; case TAG_FMT_DOUBLE: - memmove(data_ptr, &info_data->value.d, byte_count); + memmove(data_ptr, &info_value->d, 8); data_ptr += 8; break; } diff --git a/ext/exif/tests/bug68113.jpg b/ext/exif/tests/bug68113.jpg new file mode 100755 index 0000000..3ce7a62 Binary files /dev/null and b/ext/exif/tests/bug68113.jpg differ diff --git a/ext/exif/tests/bug68113.phpt b/ext/exif/tests/bug68113.phpt new file mode 100644 index 0000000..0fa4c4a --- /dev/null +++ b/ext/exif/tests/bug68113.phpt @@ -0,0 +1,17 @@ +--TEST-- +Bug #68113 (Heap corruption in exif_thumbnail()) +--SKIPIF-- +<?php +extension_loaded("exif") or die("skip need exif"); +?> +--FILE-- +<?php +var_dump(exif_thumbnail(__DIR__."/bug68113.jpg")); +?> +Done +--EXPECTF-- +Warning: exif_thumbnail(bug68113.jpg): File structure corrupted in %s/bug68113.php on line 2 + +Warning: exif_thumbnail(bug68113.jpg): Invalid JPEG file in %s/bug68113.php on line 2 +bool(false) +Done \ No newline at end of file