Skip to content

Update clampedToValidWindowSize to use Int32.max#546

Merged
gjcairo merged 1 commit into
apple:mainfrom
madsodgaard:main
Apr 14, 2026
Merged

Update clampedToValidWindowSize to use Int32.max#546
gjcairo merged 1 commit into
apple:mainfrom
madsodgaard:main

Conversation

@madsodgaard
Copy link
Copy Markdown
Contributor

Fixes a runtime crash on 32-bit devices because of an overflow.

This occurred immediately when trying to make a HTTP2 request on our Android 32-bit devices.

Copy link
Copy Markdown
Contributor

@gjcairo gjcairo left a comment

Choose a reason for hiding this comment

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

Thanks @madsodgaard!

@gjcairo gjcairo enabled auto-merge (squash) April 14, 2026 13:18
@gjcairo gjcairo added the 🔨 semver/patch No public API change. label Apr 14, 2026
@gjcairo gjcairo merged commit 81cc182 into apple:main Apr 14, 2026
55 of 57 checks passed
@madsodgaard
Copy link
Copy Markdown
Contributor Author

@gjcairo thanks! is it possible to tag a new release as well?

@gjcairo
Copy link
Copy Markdown
Contributor

gjcairo commented Apr 15, 2026

Just tagged one :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 semver/patch No public API change.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants