Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgraded the Jackson version to address CVE vulnerabilities from jackson-databind #23752

Closed

Conversation

sumi-mathew
Copy link

@sumi-mathew sumi-mathew commented Oct 1, 2024

Description

To address the CVE issues related to jackson-databind (CVE-2022-42003), upgraded the version of Jackson from 2.11.0 to 2.13.5.

Motivation and Context

The motivation for fixing CVEs in Jackson-databind primarily arises from the identification of security vulnerabilities that could be exploited by attackers. Jackson-databind is a widely used library in Java applications for processing JSON data. Given its extensive adoption, vulnerabilities in this library pose significant risks across numerous applications. Fixing these issues is crucial to protecting the broader ecosystem.

Impact

Image scan showed the vulnerability has been removed(CVE-2022-42003).
Image scan report after fix:
jacson_databind.csv

Test Plan

Contributor checklist

  • Please make sure your submission complies with our development, formatting, commit message, and attribution guidelines.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== RELEASE NOTES ==

Security
* Fix provided for the CVE-2022-42003 :pr:`23752`
* Upgrade the version of jackson from 2.11.0 to 2.13.5  :pr:`23752`

Copy link

linux-foundation-easycla bot commented Oct 1, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@sumi-mathew sumi-mathew changed the title Upgraded the Jackson version to address CVE vulnerabilities Upgraded the Jackson version to address CVE vulnerabilities from jackson-databind Oct 1, 2024
@sumi-mathew sumi-mathew marked this pull request as ready for review October 1, 2024 06:27
@elharo
Copy link
Contributor

elharo commented Oct 1, 2024

Oh joy., Another security bug in jackson. In other news, grass is green and water is wet. I wish projects, Presto included, would just stop using this library.

Copy link
Contributor

@elharo elharo left a comment

Choose a reason for hiding this comment

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

Why not a later version? e.g. 2.17.1?

@sumi-mathew sumi-mathew force-pushed the jackson-databind-cve branch 3 times, most recently from 5c36e08 to 5d3181b Compare October 3, 2024 06:13
@sumi-mathew
Copy link
Author

Resolved as part of PR: #23753

@sumi-mathew sumi-mathew closed this Oct 7, 2024
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.

2 participants