Bug #72735 [Csd->Fbk]: 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: Closed +Status: Feedback Type: Bug Package: EXIF related Operating System: Mageia5 PHP Version: 5.6.24 -Assigned To: +Assigned To: kalle Block user comment: N Private report: N New Comment: 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! Previous Comments: ------------------------------------------------------------------------ [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 ------------------------------------------------------------------------ [2016-08-02 16:07:07] kalle@php.net Any chance you can leave a link to the pictures here in case a developer decides to pick it up? ------------------------------------------------------------------------ [2016-08-02 14:33:07] antonin dot roussel at laposte dot net Description: ------------ Extracting thumbnail of pictures in a gallery is ok (for instance nikon camera pictures), but for Samsung ST72/ST73 camera pictures it's broken. The data return by exif_thumbnail php function has a zero strlentgh. To cross-check : command line exiftool sees exif preview and thumbnail data. So there is a problem. I can provide two sample pictures. ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=72735&edit=1