Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update exif.h #2101

Merged
merged 4 commits into from Feb 1, 2023
Merged

Update exif.h #2101

merged 4 commits into from Feb 1, 2023

Conversation

a-shvedov
Copy link
Contributor

trying to fix #2100

@google-cla
Copy link

google-cla bot commented Jan 24, 2023

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.

lib/jxl/exif.h Outdated Show resolved Hide resolved
@mo271
Copy link
Member

mo271 commented Jan 31, 2023

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

@a-shvedov
Copy link
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
lib/jxl/exif.h Outdated Show resolved Hide resolved
a-shvedov and others added 4 commits February 1, 2023 09:31
Co-authored-by: Moritz Firsching <firsching@google.com>
@mo271 mo271 merged commit a7c8428 into libjxl:main Feb 1, 2023
62 checks passed
@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>
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