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=9c8c37854efe843e45228597837fc4b6aac59113 Log: Fixed bug #72735 (Samsung picture thumb not read (zero size)) Previous Comments: ------------------------------------------------------------------------ [2016-08-03 05:11:26] remi@php.net Could be related to recent security fix (#50845, #72603, #72618) Also reported as https://bugzilla.redhat.com/1362571 (with Panasonic samples) ------------------------------------------------------------------------ [2016-08-03 04:11:12] kalle@php.net 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))" ------------------------------------------------------------------------ [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! ------------------------------------------------------------------------ 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