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

Address some initial fuzz reports after core rewrite merge #1821

Merged

Conversation

kdt3rd
Copy link
Contributor

@kdt3rd kdt3rd commented Sep 16, 2024

This will handle OSS-Fuzz report IDs 71610, 71612, 71615

… that they exist

Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>
… memory sanitizer warning

Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>
Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>
Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>
Copy link
Member

@cary-ilm cary-ilm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cary-ilm cary-ilm merged commit 90704f2 into AcademySoftwareFoundation:main Sep 16, 2024
34 checks passed
@kdt3rd kdt3rd deleted the fix_ossfuzz_core_cpp_rewrite_1 branch September 16, 2024 20:29
asa pushed a commit to asa/openexr that referenced this pull request Sep 21, 2024
…ftwareFoundation#1821)

* Add validation that the types of the attributes are correct, not just that they exist

Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>

* use constructor semantics to zero init the context pointer, avoid one memory sanitizer warning

Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>

* to avoid UB with streams, keep attr sizes with int32 size

Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>

* handle an empty buffer cleanly

Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>

---------

Signed-off-by: Kimball Thurston <kdt3rd@gmail.com>
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