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 User updated by: antonin dot roussel at laposte dot 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: you misunderstood me, I was just inflecting my comment about added sample picture sony_dsc-rx100.jpg which indeed appears as broken as others do. I cannot evaluate patches. Thank you for making php handy ;-) Previous Comments: ------------------------------------------------------------------------ [2016-08-03 14:50:12] kalle@php.net @Remi, one thing I was thinking about your patch, should we add a notice/warning in case we run out of possible signatures to match against, urging users to submit a bug report so we can cease out these types of reports? ------------------------------------------------------------------------ [2016-08-03 14:40:07] kalle@php.net @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!) ------------------------------------------------------------------------ [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)); ------------------------------------------------------------------------ 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