]> 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)
committerDavey Shafik <me@daveyshafik.com>
Wed, 3 Aug 2016 19:29:03 +0000 (12:29 -0700)
commit835ef14979c6953cbe9b4ead58649c587ccfe2ba
treec189fdecd16ae7316c96f155088476cda5a25d15
parent6822af2e738698aab9fa34ca0c572309960bcd74
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.
ext/exif/exif.c