Skip to content

Check that size is non-negative when reading string or bytes in Strea…#17514

Merged
zhangskz merged 2 commits into25.xfrom
cp-25
Jul 17, 2024
Merged

Check that size is non-negative when reading string or bytes in Strea…#17514
zhangskz merged 2 commits into25.xfrom
cp-25

Conversation

@zhangskz
Copy link
Copy Markdown
Member

@zhangskz zhangskz commented Jul 17, 2024

…mDecoder.

This ensures that StreamDecoder throws a InvalidProtocolBufferException instead of an IllegalStateException on some invalid input.

All other implementations of CodedInputStream already do this check.

PiperOrigin-RevId: 623383287

Cherry-pick of a94f57b

…mDecoder.

This ensures that StreamDecoder throws a InvalidProtocolBufferException instead of an IllegalStateException on some invalid input.

All other implementations of CodedInputStream already do this check.

PiperOrigin-RevId: 623383287
@zhangskz zhangskz requested a review from a team as a code owner July 17, 2024 20:19
@zhangskz zhangskz requested review from mkruskal-google and shaod2 and removed request for a team July 17, 2024 20:19
This was fixed in abseil/abseil-cpp@cd7f66c, but older versions of Abseil break in CMake 3.30, which github runners now use by default.

PiperOrigin-RevId: 651854867
@zhangskz zhangskz requested a review from a team as a code owner July 17, 2024 20:41
@zhangskz zhangskz merged commit fb0520e into 25.x Jul 17, 2024
@zhangskz zhangskz deleted the cp-25 branch July 17, 2024 21:45
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.

4 participants