Skip to content

Update exif.h - #2101

Merged
mo271 merged 4 commits into
libjxl:mainfrom
sigdevel:main
Feb 1, 2023
Merged

mo271 merged 4 commits into
libjxl:mainfrom
sigdevel:main

Conversation

@sigdevel

Copy link
Copy Markdown
Contributor

trying to fix #2100

@google-cla

google-cla Bot commented Jan 24, 2023

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Comment thread lib/jxl/exif.h Outdated
@mo271

mo271 commented Jan 31, 2023

Copy link
Copy Markdown
Member

Thanks for the suggested change, @a-shvedov! Could you follow the instructions above from the google-cla bot?!

@sigdevel

Copy link
Copy Markdown
Contributor Author

Thanks for the suggested change, @a-shvedov! Could you follow the instructions above from the google-cla bot?!

Yes, sure!

@mo271
mo271 requested a review from veluca93 January 31, 2023 22:24
Comment thread lib/jxl/exif.h Outdated
sigdevel and others added 4 commits February 1, 2023 09:31
Co-authored-by: Moritz Firsching <firsching@google.com>
@mo271
mo271 enabled auto-merge (squash) February 1, 2023 08:32
@mo271
mo271 merged commit a7c8428 into libjxl:main Feb 1, 2023
@mo271 mo271 added the merge-0.8 label Feb 1, 2023
mo271 pushed a commit to mo271/libjxl that referenced this pull request Feb 1, 2023
* Update exif.h

* Update lib/jxl/exif.h

Co-authored-by: Moritz Firsching <firsching@google.com>

* fix AUTHORS

* fix lint

---------

Co-authored-by: Moritz Firsching <firsching@google.com>
(cherry picked from commit a7c8428)

PR libjxl#2101
eustas pushed a commit that referenced this pull request Feb 1, 2023
* Update exif.h

* Update lib/jxl/exif.h

Co-authored-by: Moritz Firsching <firsching@google.com>

* fix AUTHORS

* fix lint

---------

Co-authored-by: Moritz Firsching <firsching@google.com>
(cherry picked from commit a7c8428)

PR #2101
jonsneyers pushed a commit to jonsneyers/libjxl that referenced this pull request Apr 24, 2023
* Update exif.h

* Update lib/jxl/exif.h

Co-authored-by: Moritz Firsching <firsching@google.com>

* fix AUTHORS

* fix lint

---------

Co-authored-by: Moritz Firsching <firsching@google.com>
sezero pushed a commit to libsdl-org/libjxl that referenced this pull request Aug 17, 2025
* Update exif.h

* Update lib/jxl/exif.h

Co-authored-by: Moritz Firsching <firsching@google.com>

* fix AUTHORS

* fix lint

---------

Co-authored-by: Moritz Firsching <firsching@google.com>

(cherry picked from commit a7c8428
 for CVE-2023-0645: out of bounds read in the exif handler)
sezero pushed a commit to libsdl-org/libjxl that referenced this pull request Feb 10, 2026
* Update exif.h

* Update lib/jxl/exif.h

Co-authored-by: Moritz Firsching <firsching@google.com>

* fix AUTHORS

* fix lint

---------

Co-authored-by: Moritz Firsching <firsching@google.com>

(cherry picked from commit a7c8428
 for CVE-2023-0645: out of bounds read in the exif handler)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

jxl::FindExifTagPosition - pointing to memory that is out of bounds (Segfault)

3 participants