Bug #72735 [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: 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: @antonin, can you open another report and assign it to me and I will take a look at it, just reference this as it makes the whole process a little easier to deal with. Thanks (and thanks for helping PHP become even greater!) Previous Comments: ------------------------------------------------------------------------ [2016-08-03 12:52:50] antonin dot roussel at laposte dot net sorry, there is no "slightly different aspect" ------------------------------------------------------------------------ [2016-08-03 08:38:47] remi@php.net 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 ------------------------------------------------------------------------ [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. ------------------------------------------------------------------------ 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