]> granicus.if.org Git - php/commit
Changed the way MAKERNOTE is handled in case we do not have a matching signature...
authorKalle Sommer Nielsen <kalle@php.net>
Wed, 3 Aug 2016 15:05:31 +0000 (17:05 +0200)
committerAnatol Belski <ab@php.net>
Wed, 3 Aug 2016 15:36:00 +0000 (17:36 +0200)
commitd5796fb298abb2a3e389818ad924589fa35e58b9
tree2fb6d066124fc564099bc3d836ae8a1b3eedadf6
parent67c2b0a9e8c746fca5bd0ef17e8d184c4f201020
Changed the way MAKERNOTE is handled in case we do not have a matching signature (Remi).

Before this patch, exif_process_IFD_in_MAKERNOTE() would return false, then causing the rest of the EXIF parsing to be interrupted. This is a regression from earlier which was most likely a part of a security fix for MAKERNOTE.

The new behavior is to instead of stopping to parse, to continue so we can still fetch data like thumbnail and GPS, thrus allowing yet unsupported formats to parse. If EXIF's debugging mode is enabled, a notice will display in case we do not match against a valid MAKERNOTE signature.

This should temporarily fix bug #72682 (exif_read_data() fails to read all data for some images) until I get around to debug it further.

(cherry picked from commit aabcb5481d9e717df77192dab2894468b9fc63b4)
ext/exif/exif.c