Skip to content

Add check for span length in parse PNG pHYs chunk#3080

Merged
JimBobSquarePants merged 4 commits intomainfrom
bp/fixIssue3078
Mar 10, 2026
Merged

Add check for span length in parse PNG pHYs chunk#3080
JimBobSquarePants merged 4 commits intomainfrom
bp/fixIssue3078

Conversation

@brianpopow
Copy link
Collaborator

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following the existing coding patterns and practice as demonstrated in the repository. These follow strict Stylecop rules 👮.
  • I have provided test coverage for my change (where applicable)

Description

This PR fixes an issue when parsing the pHYs when not enough data is present described in #3078

Copy link
Member

@JimBobSquarePants JimBobSquarePants left a comment

Choose a reason for hiding this comment

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

Perfect. Thanks for looking after this!

@JimBobSquarePants JimBobSquarePants linked an issue Mar 10, 2026 that may be closed by this pull request
4 tasks
@JimBobSquarePants JimBobSquarePants merged commit 65dd8bd into main Mar 10, 2026
12 checks passed
@JimBobSquarePants JimBobSquarePants deleted the bp/fixIssue3078 branch March 10, 2026 02:46
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.

IndexOutOfRangeException in PngPhysical.Parse with truncated pHYs chunk

2 participants