]> granicus.if.org Git - php/commit
Fix #71848: getimagesize with $imageinfo returns false
authorChristoph M. Becker <cmbecker69@gmx.de>
Sat, 30 Jun 2018 14:29:29 +0000 (16:29 +0200)
committerChristoph M. Becker <cmbecker69@gmx.de>
Sat, 30 Jun 2018 14:29:30 +0000 (16:29 +0200)
commitae04110032702622d59c21f9e615120d9479157a
tree3990f71362820f2479eaed364d5546d4be723c73
parent962706d16c29ec6dd5ec3a78edae9e8907bca20a
Fix #71848: getimagesize with $imageinfo returns false

Some JFIF images contain empty APP segments, i.e. those which consist
only of the marker bytes and the length, but without actual content.
It appears to be doubtful to have empty APP segments, but we should
apply the robustness principle, and accept these, instead of simply
failing in this case.

We choose to add empty APP segments to $imageinfo with an empty string
as value, instead of NULL, or even to omit these segments altogether.

This patch also fixes the potential issue that php_stream_read() might
not read the supposed number of bytes, which could result in garbage to
be added to the read value.
NEWS
ext/standard/image.c
ext/standard/tests/image/bug71848.jpg [new file with mode: 0644]
ext/standard/tests/image/bug71848.phpt [new file with mode: 0644]