Bug #72735 [Asn->Csd]: Samsung picture thumb not read (zero size)

Edit report at https://bugs.php.net/bug.php?id=72735&edit=1 ID: 72735 Updated by: kalle@php.net Reported by: antonin dot roussel at laposte dot net Summary: Samsung picture thumb not read (zero size) -Status: Assigned +Status: Closed Type: Bug Package: EXIF related Operating System: Mageia5 PHP Version: 5.6.24 Assigned To: kalle Block user comment: N Private report: N New Comment: Automatic comment on behalf of kalle Revision: http://git.php.net/?p=php-src.git;a=commit;h=aa821d8cad082634c554c6a5ed9233cdc24b9cff Log: Revert "Fixed bug #72735 (Samsung picture thumb not read (zero size))" Previous Comments: ------------------------------------------------------------------------ [2016-08-03 04:10:17] kalle@php.net I've reverted the fix since it seems to break other tests within ext/exif, so I will need a tiny bit more debugging to make sure it will work. Apologies for the crushed dreams /o\ ------------------------------------------------------------------------ [2016-08-03 03:55:21] kalle@php.net Related To: Bug #55420 ------------------------------------------------------------------------ [2016-08-03 03:42:36] kalle@php.net I committed a possible fix to this on master. In my debugging it seems that there were 2 IDF tags in samsung.jpg that were returning invalid, that stopped processing (more description are available in the commit message). Could you please test and confirm on your end? Ps. its master in git, which is PHP 7.2, thanks! ------------------------------------------------------------------------ [2016-08-03 03:40:18] kalle@php.net Automatic comment on behalf of kalle Revision: http://git.php.net/?p=php-src.git;a=commit;h=406dff2a02ff1cfac89942ce5e991d88d3b6cb50 Log: Fixed bug #72735 (Samsung picture thumb not read (zero size)) ------------------------------------------------------------------------ [2016-08-02 22:08:02] antonin dot roussel at laposte dot net Here are some sample files, and a test script. 'sam4068.jpg', // broken (it's coming from a friend) 'samsung.jpg', // * broken (minimized) 'dscn5460.jpg', // * ok (from my camera) 'nokia.jpg', // * ok (minimized) // * star means : have been processed by exiftool and convert https://www.dropbox.com/sh/ru4kwqsyt9fqd89/AACTpWWpBojkNuYGkFgsLWj2a?dl=0 ------------------------------------------------------------------------ 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=72735 -- Edit this bug report at https://bugs.php.net/bug.php?id=72735&edit=1