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

[GHSA-898j-5cc8-cmf5] Moderate severity vulnerability that affects org.apache.storm:storm-core #3838

Conversation

MarkLee131
Copy link

Updates

  • References

Comments
Add four patfches:
apache/storm@0fc6b52
apache/storm@efad4cc
apache/storm@1117a37
apache/storm@f61e5da

, of which the commit message claims STORM-3052: Allow for blobs to be unzipped/untarred

@github-actions github-actions bot changed the base branch from main to MarkLee131/advisory-improvement-3838 March 3, 2024 17:41
@darakian
Copy link
Contributor

darakian commented Mar 4, 2024

Hey @MarkLee131, not sure I follow on this one. Can you elaborate on how the commits resolve a zip slip and/or do you know of some linkage to the advisory?

@MarkLee131
Copy link
Author

Hi, you can see the root cause of this CVE on the NVD or CVE website. and the commit msgs and code diff of these commits show their intention for fixing the zip-slip vuln for archive files in STORM-3052: https://issues.apache.org/jira/browse/STORM-3052.

Meanwhile, Synk also detailed this vuln in its report (https://security.snyk.io/vuln/SNYK-JAVA-ORGAPACHESTORM-73624), and referred to one of our updated patch commit: apache/storm@1117a37.

@advisory-database advisory-database bot merged commit a54cdb0 into MarkLee131/advisory-improvement-3838 Mar 20, 2024
2 checks passed
@advisory-database advisory-database bot deleted the MarkLee131-GHSA-898j-5cc8-cmf5 branch March 20, 2024 17:34
@advisory-database
Copy link
Contributor

Hi @MarkLee131! Thank you so much for contributing to the GitHub Advisory Database. This database is free, open, and accessible to all, and it's people like you who make it great. Thanks for choosing to help others. We hope you send in more contributions in the future!

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