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-hmhq-382q-mp56] ClassLoader manipulation in Apache Struts #4241

Conversation

MarkLee131
Copy link

Updates

  • Affected products
  • Description
  • References

Comments
Add a ""potential patch" commit apache/struts@74e2683, of which the commit message claims Adds additional pattern to prevent access to getClass method, this commit tried to fix CVE-2014-0112 and CVE-2014-0113, as the previous pr I opened. As described in the CVE desc of CVE-2014-0116, this commit would provide some insights for this vuln, even though it may be not a valid patch commit for this cve.

@github-actions github-actions bot changed the base branch from main to MarkLee131/advisory-improvement-4241 April 12, 2024 06:06
@shelbyc
Copy link
Contributor

shelbyc commented Apr 16, 2024

Hi @MarkLee131, similar to #4239 and #4240, apache/struts@74e2683 was committed on 10 January 2016 and GHSA-prjv-jj26-wf8h was patched in 2014. This makes apache/struts@74e2683 too new to be the fix commit.

@taladrane
Copy link
Collaborator

👋 This pull request has been marked as stale because it has been open with no activity. You can: comment on the issue or remove the stale label to hold stale off for a while, add the Keep label to hold stale off permanently, or do nothing. If you do nothing this pull request will be closed eventually by the stale bot. Please see CONTRIBUTING.md for more policy details.

@taladrane taladrane added Stale and removed Stale labels May 2, 2024
@MarkLee131 MarkLee131 closed this May 15, 2024
@github-actions github-actions bot deleted the MarkLee131-GHSA-hmhq-382q-mp56 branch May 15, 2024 06:17
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.

3 participants