Skip to content

Reject NO_COMPRESSION scanline chunks with short packed size.#2548

Merged
cary-ilm merged 2 commits into
AcademySoftwareFoundation:mainfrom
cary-ilm:fix-GHSA-none-scanline-short
Jul 14, 2026
Merged

Reject NO_COMPRESSION scanline chunks with short packed size.#2548
cary-ilm merged 2 commits into
AcademySoftwareFoundation:mainfrom
cary-ilm:fix-GHSA-none-scanline-short

Conversation

@cary-ilm

Copy link
Copy Markdown
Member

Mirror the existing tile-path check in exr_read_scanline_chunk_info(): for EXR_COMPRESSION_NONE, require packed_size to equal unpacked_size before decode allocates an unpack buffer that no decompressor fills.

Covers flat and deep scanline chunk leaders.

Addresses https://github.com/AcademySoftwareFoundation/openexr/security/advisories/GHSA-7hgm-jxjc-6hgg

Addresses https://github.com/AcademySoftwareFoundation/openexr/security/advisories/GHSA-fwhm-jj7c-mx7v

Mirror the existing tile-path check in exr_read_scanline_chunk_info():
for EXR_COMPRESSION_NONE, require packed_size to equal unpacked_size
before decode allocates an unpack buffer that no decompressor fills.

Covers flat and deep scanline chunk leaders.

Addresses https://github.com/AcademySoftwareFoundation/openexr/security/advisories/GHSA-7hgm-jxjc-6hgg

Addresses https://github.com/AcademySoftwareFoundation/openexr/security/advisories/GHSA-fwhm-jj7c-mx7v

Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: Cary Phillips <seabeepea@gmail.com>

@peterhillman peterhillman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good

@cary-ilm
cary-ilm merged commit c07a0f1 into AcademySoftwareFoundation:main Jul 14, 2026
50 of 51 checks passed
@cary-ilm cary-ilm added the v3.4 label Jul 14, 2026
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.

2 participants