Skip to content

fix(security): [Aikido] [SEC-725] Fix 2 security issues in google.golang.org/protobuf, x/sys - #165

Open
aikido-autofix[bot] wants to merge 3 commits into
mainfrom
fix/aikido-security-update-packages-63038889-j5c7
Open

fix(security): [Aikido] [SEC-725] Fix 2 security issues in google.golang.org/protobuf, x/sys#165
aikido-autofix[bot] wants to merge 3 commits into
mainfrom
fix/aikido-security-update-packages-63038889-j5c7

Conversation

@aikido-autofix

@aikido-autofix aikido-autofix Bot commented Jul 9, 2026

Copy link
Copy Markdown

Upgrade protobuf and sys to fix infinite loop DoS in protojson.Unmarshal and string overflow handling vulnerability.

✅ 2 CVEs resolved by this upgrade

This PR will resolve the following CVEs:

Issue Severity           Description
CVE-2024-24786
HIGH
[google.golang.org/protobuf] The protojson.Unmarshal function can enter an infinite loop when unmarshaling certain forms of invalid JSON. This condition can occur when unmarshaling into a message which contains a google.protobuf.Any value, or when the UnmarshalOptions.DiscardUnknown option is set.
CVE-2026-39824
LOW
[golang.org/x/sys] NewNTUnicodeString does not check for string length overflow. When provided with a string that overflows the maximum size of a NTUnicodeString (a 16-bit number of bytes), it returns a truncated string rather than an error.

@sonarqubecloud

Copy link
Copy Markdown

@ChristianWitts ChristianWitts 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.

Approving Aikido security autofix — CI is green. Owning team: please give this a final review and merge.

@ChristianWitts ChristianWitts changed the title fix(security): [Aikido] [] Fix 2 security issues in google.golang.org/protobuf, x/sys fix(security): [Aikido] [SEC-725] Fix 2 security issues in google.golang.org/protobuf, x/sys Jul 27, 2026

@kelu-diao kelu-diao left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approving as part of the batch PR review requested in Slack — CI checks are green. Owning team: please do the final review and merge.


Generated by Claude Code

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