Bug #72682 [Com]: exif_read_data() fails to read all data for some images

Edit report at https://bugs.php.net/bug.php?id=72682&edit=1 ID: 72682 Comment by: php at barryhunter dot co dot uk Reported by: r dot helm at helm-software dot de Summary: exif_read_data() fails to read all data for some images Status: Closed Type: Bug Package: EXIF related PHP Version: 7.0.9 Assigned To: kalle Block user comment: N Private report: N New Comment: Think have the same issue, and have a test case that still fails: http://cake.geograph.org.uk/exif/_exif_test-andrew.php This image: http://cake.geograph.org.uk/exif/IMG_6216.org.JPG still fails to read completely. Stipping the makernote from the image (via exiftool), allows the full data to be see by exif_read_data function. Its a image from a Canon camera. This is against 5.6.28, but the original issues, and potential fix appear to be backported. Previous Comments: ------------------------------------------------------------------------ [2016-10-10 11:17:47] krakjoe@php.net Automatic comment on behalf of kalle Revision: http://git.php.net/?p=php-src.git;a=commit;h=c6bd054b86c52948505be7409ad8d6488db062f6 Log: Fixed bug #72682 (exif_read_data() fails to read all data for some images) ------------------------------------------------------------------------ [2016-08-22 07:44:01] sina dot raudszus at heg dot com as stated in the last comment from [2016-08-03 13:51 UTC] it is not fixed. we've got customers who complain about it. could you do something about this please? ------------------------------------------------------------------------ [2016-08-03 15:33:34] kalle@php.net Automatic comment on behalf of kalle Revision: http://git.php.net/?p=php-src.git;a=commit;h=c6bd054b86c52948505be7409ad8d6488db062f6 Log: Fixed bug #72682 (exif_read_data() fails to read all data for some images) ------------------------------------------------------------------------ [2016-08-03 13:51:09] r dot helm at helm-software dot de It's not fixed. Take a look at my previous comment. In exif_process_IFD_in_MAKERNOTE() it goes through maker_note_array to see if it knowns how to parse the makernote of this image. If it doesn't find anything it returns FALSE, which was probably intended to mean that it did not process the makernote. Previously the result of this function was ignored but now when it returns false parsing is stopped. So i think it should return TRUE. ------------------------------------------------------------------------ [2016-08-03 10:02:49] kalle@php.net Please try using this snapshot: http://snaps.php.net/php-trunk-latest.tar.gz For Windows: http://windows.php.net/snapshots/ Gonna steal this one from you Stas. Me and @remi are working on a patch to hopefully resolve this, along with a few other EXIF related bugs at the same time. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=72682 -- Edit this bug report at https://bugs.php.net/bug.php?id=72682&edit=1