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

Overflowing a short length so it goes negative and screws up the byte… #524

Merged
merged 1 commit into from
Jan 12, 2021

Conversation

sergeyplokhikh
Copy link
Contributor

… read pointer

@drewnoakes
Copy link
Owner

@sergeyplokhikh thanks for this. Do you have an image that exhibits this problem that you can share with the project?

@sergeyplokhikh
Copy link
Contributor Author

@sergeyplokhikh thanks for this. Do you have an image that exhibits this problem that you can share with the project?

Images I have are a customers so we can’t share directly but I'll see if I can sanitize one.

@drewnoakes drewnoakes merged commit 57a5fa6 into drewnoakes:master Jan 12, 2021
@sergeyplokhikh
Copy link
Contributor Author

@drewnoakes
Copy link
Owner

@sergeyplokhikh thanks very much for following up with the file. Appreciate it.

drewnoakes added a commit to drewnoakes/metadata-extractor-images that referenced this pull request Jan 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants