diff options
author | Gabor Buella <gbuella@gmail.com> | 2014-04-05 00:17:25 +0200 |
---|---|---|
committer | Stanislav Malyshev <stas@php.net> | 2014-04-13 15:17:04 -0700 |
commit | 1010200da5da81642bec89d9c90d001478e3554d (patch) | |
tree | 5edabf6b76d699551682718d5af88e5345a69380 /ext/standard | |
parent | beda5093b490e80f631f14555bba7216c284a2c3 (diff) | |
download | php-git-1010200da5da81642bec89d9c90d001478e3554d.tar.gz |
Fixed bug #67024 - getimagesize should recognize BMP files with negative height
Diffstat (limited to 'ext/standard')
-rw-r--r-- | ext/standard/image.c | 1 | ||||
-rw-r--r-- | ext/standard/tests/image/getimagesize.phpt | 17 | ||||
-rw-r--r-- | ext/standard/tests/image/image_type_to_mime_type.phpt | 4 | ||||
-rw-r--r-- | ext/standard/tests/image/test-1pix.bmp | bin | 0 -> 58 bytes |
4 files changed, 20 insertions, 2 deletions
diff --git a/ext/standard/image.c b/ext/standard/image.c index 27f60c95a7..816de44d4c 100644 --- a/ext/standard/image.c +++ b/ext/standard/image.c @@ -163,6 +163,7 @@ static struct gfxinfo *php_handle_bmp (php_stream * stream TSRMLS_DC) result = (struct gfxinfo *) ecalloc (1, sizeof(struct gfxinfo)); result->width = (((unsigned int)dim[ 7]) << 24) + (((unsigned int)dim[ 6]) << 16) + (((unsigned int)dim[ 5]) << 8) + ((unsigned int) dim[ 4]); result->height = (((unsigned int)dim[11]) << 24) + (((unsigned int)dim[10]) << 16) + (((unsigned int)dim[ 9]) << 8) + ((unsigned int) dim[ 8]); + result->height = abs((int32_t)result->height); result->bits = (((unsigned int)dim[15]) << 8) + ((unsigned int)dim[14]); } else { return NULL; diff --git a/ext/standard/tests/image/getimagesize.phpt b/ext/standard/tests/image/getimagesize.phpt index 6cd8275e06..04ddd8c82b 100644 --- a/ext/standard/tests/image/getimagesize.phpt +++ b/ext/standard/tests/image/getimagesize.phpt @@ -23,7 +23,22 @@ GetImageSize() var_dump($result); ?> --EXPECT-- -array(12) { +array(13) { + ["test-1pix.bmp"]=> + array(6) { + [0]=> + int(1) + [1]=> + int(1) + [2]=> + int(6) + [3]=> + string(20) "width="1" height="1"" + ["bits"]=> + int(24) + ["mime"]=> + string(14) "image/x-ms-bmp" + } ["test1bpix.bmp"]=> array(6) { [0]=> diff --git a/ext/standard/tests/image/image_type_to_mime_type.phpt b/ext/standard/tests/image/image_type_to_mime_type.phpt index d83ab8d146..9f7ffa1aa3 100644 --- a/ext/standard/tests/image/image_type_to_mime_type.phpt +++ b/ext/standard/tests/image/image_type_to_mime_type.phpt @@ -25,7 +25,9 @@ image_type_to_mime_type() var_dump($result); ?> --EXPECT-- -array(12) { +array(13) { + ["test-1pix.bmp"]=> + string(14) "image/x-ms-bmp" ["test1bpix.bmp"]=> string(14) "image/x-ms-bmp" ["test1pix.bmp"]=> diff --git a/ext/standard/tests/image/test-1pix.bmp b/ext/standard/tests/image/test-1pix.bmp Binary files differnew file mode 100644 index 0000000000..ec68a571cc --- /dev/null +++ b/ext/standard/tests/image/test-1pix.bmp |