Skip to content

Upgrade snappy-java to 1.1.10.1 to fix CVE-2023-34453#1614

Open
sidhdirenge wants to merge 2 commits into
developfrom
jetski/fix-cve-2023-34453-39410
Open

Upgrade snappy-java to 1.1.10.1 to fix CVE-2023-34453#1614
sidhdirenge wants to merge 2 commits into
developfrom
jetski/fix-cve-2023-34453-39410

Conversation

@sidhdirenge

Copy link
Copy Markdown

This PR upgrades snappy-java to 1.1.10.1 to remediate CVE-2023-34453. Note: avro is already at version 1.11.4 in develop branch.

@sidhdirenge sidhdirenge added the build Trigger unit test build label Jun 17, 2026

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request updates the snappy-java dependency version in pom.xml from 1.1.7.2 to 1.1.10.1. While this upgrade addresses CVE-2023-34453, the selected version is still vulnerable to CVE-2023-43642. It is recommended to upgrade to version 1.1.10.5 instead to ensure all critical security vulnerabilities are resolved.

Comment thread pom.xml Outdated
<groupId>org.xerial.snappy</groupId>
<artifactId>snappy-java</artifactId>
<version>1.1.7.2</version>
<version>1.1.10.1</version>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

security-critical critical

While upgrading to 1.1.10.1 remediates CVE-2023-34453, this version remains vulnerable to CVE-2023-43642, which is a critical vulnerability where a missing upper bound check on chunk length can lead to java.lang.OutOfMemoryError or integer overflow. It is highly recommended to upgrade to 1.1.10.5 (or at least 1.1.10.4) to resolve both security issues.

Suggested change
<version>1.1.10.1</version>
<version>1.1.10.5</version>

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

Labels

build Trigger unit test build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant