diff options
author | Tobias Stoeckmann <stoeckmann@users.noreply.github.com> | 2021-07-23 20:17:50 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-07-23 20:17:50 +0200 |
commit | c188578f3f807c967b54b73b793399f0e23f54b3 (patch) | |
tree | 1d7e827e65c57452f1b35ce7516bfd826c6993c7 /src/options.h | |
parent | 87d9b7edb0e348c606d0d064e8db998ee9cec208 (diff) |
Fix out of boundary access with exif (#617)
* Fix out of boundary access
The while-loop is not finished when pos is set to EXIF_MAX_DATA.
Instead, the loop continues and therefore tries to access data outside
of the array.
This is triggered when compiled with exif=1 and asan:
$ feh --draw-exif image.jpg
* Fixed formatting
No functional change but makes previous commit easier verifiable
(independent of tab space setup).
* Call break; instead of setting pos2 to a magic value
This is in line with the following else clause
* Another cosmetic adjustment
Co-authored-by: Daniel Friesel <derf@finalrewind.org>
Diffstat (limited to 'src/options.h')
0 files changed, 0 insertions, 0 deletions