Bug #72735 [PATCH]: Samsung picture thumb not read (zero size)

Edit report at https://bugs.php.net/bug.php?id=72735&edit=1 ID: 72735 Patch added by: remi@php.net Reported by: antonin dot roussel at laposte dot net Summary: Samsung picture thumb not read (zero size) 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: The following patch has been added/updated: Patch Name: 0001-Fix-72735-regression-in-exif-for-MakerNote.patch Revision: 1470213526 URL: https://bugs.php.net/patch-display.php?bug=72735&patch=0001-Fix-72735-regression-in-exif-for-MakerNote.patch&revision=1470213526 Previous Comments: ------------------------------------------------------------------------ [2016-08-03 08:34:19] remi@php.net Probably, should be length = MIN(length, strlen(value)); ------------------------------------------------------------------------ [2016-08-03 08:27:25] remi@php.net @Kalle: your patch is fine, but only for SAMSUNG, Fedora bug is about Panasonic, please give a try to my complementary fix proposal. ------------------------------------------------------------------------ [2016-08-03 08:25:43] antonin dot roussel at laposte dot net I do not have php development environment, I won't be able to test any patch. I added two other sample pictures, from two sony camera. One is ok (sony_dslr-a700.jpg), the second (sony_dsc-rx100.jpg) shows a slightly different broken aspect. ------------------------------------------------------------------------ [2016-08-03 08:10:12] remi@php.net The following patch has been added/updated: Patch Name: 0001-Fix-72735-regression-in-exif-for-MakerNote.patch Revision: 1470211812 URL: https://bugs.php.net/patch-display.php?bug=72735&patch=0001-Fix-72735-regression-in-exif-for-MakerNote.patch&revision=1470211812 ------------------------------------------------------------------------ [2016-08-03 08:06:19] kalle@php.net @remi, can you confirm this patch also fixes the other issue from Redhat? ------------------------------------------------------------------------ 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