Skip to content

Reject NULL bytes type_hint and data in exr_attr_bytes_create()#2470

Merged
cary-ilm merged 2 commits into
AcademySoftwareFoundation:mainfrom
cary-ilm:GHSA-xx72-f24p-cf6r
Jun 17, 2026
Merged

Reject NULL bytes type_hint and data in exr_attr_bytes_create()#2470
cary-ilm merged 2 commits into
AcademySoftwareFoundation:mainfrom
cary-ilm:GHSA-xx72-f24p-cf6r

Conversation

@cary-ilm

@cary-ilm cary-ilm commented Jun 12, 2026

Copy link
Copy Markdown
Member

Validate non-null pointers when hint length or data size is non-zero, and apply the same checks in the in-place exr_attr_set_bytes() path.

Addresses https://github.com/AcademySoftwareFoundation/openexr/security/advisories/GHSA-xx72-f24p-cf6r

@cary-ilm cary-ilm requested a review from peterhillman June 12, 2026 13:31
@cary-ilm

Copy link
Copy Markdown
Member Author

@barnabyrobson, can you take a look at this?

Validate non-null pointers when hint length or data size is non-zero,
and apply the same checks in the in-place exr_attr_set_bytes() path.

Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Cary Phillips <cary@ilm.com>
@cary-ilm cary-ilm force-pushed the GHSA-xx72-f24p-cf6r branch from 404abdd to 5796d52 Compare June 12, 2026 13:32
@barnabyrobson

Copy link
Copy Markdown
Contributor

@barnabyrobson, can you take a look at this?

Looks good to me !

@cary-ilm

Copy link
Copy Markdown
Member Author

Addresses CVE-2026-55371

@kthurston kthurston left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM - thanks

@cary-ilm cary-ilm merged commit 808e2eb into AcademySoftwareFoundation:main Jun 17, 2026
49 of 51 checks passed
cary-ilm added a commit that referenced this pull request Jun 17, 2026
Validate non-null pointers when hint length or data size is non-zero,
and apply the same checks in the in-place exr_attr_set_bytes() path.

Signed-off-by: Cary Phillips <cary@ilm.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants