Skip to content

[GHSA-6q32-hq47-5qq3] @actions/artifact has an Arbitrary File Write via artifact extraction#4752

Merged
advisory-database[bot] merged 1 commit intoJLHwung/advisory-improvement-4752from
JLHwung-GHSA-6q32-hq47-5qq3
Sep 4, 2024
Merged

[GHSA-6q32-hq47-5qq3] @actions/artifact has an Arbitrary File Write via artifact extraction#4752
advisory-database[bot] merged 1 commit intoJLHwung/advisory-improvement-4752from
JLHwung-GHSA-6q32-hq47-5qq3

Conversation

@JLHwung
Copy link
Copy Markdown

@JLHwung JLHwung commented Sep 4, 2024

Updates

  • Affected products

Comments
The @actions/artifact v1 does not depend on unzip-stream v0.3.1, which contains the vulnerability1 , so the v1 version, used by both @actions/upload-artifact v3 and @actions/download-artifact v3, is not affected by this CVE.

@github
Copy link
Copy Markdown
Collaborator

github commented Sep 4, 2024

Hi there @joshmgross! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.

This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

@github-actions github-actions bot changed the base branch from main to JLHwung/advisory-improvement-4752 September 4, 2024 01:25
},
{
"type": "CVSS_V4",
"score": "CVSS:4.0/AV:N/AC:L/AT:N/PR:L/UI:N/VC:H/VI:H/VA:N/SC:N/SI:N/SA:N"
Copy link
Copy Markdown
Author

@JLHwung JLHwung Sep 4, 2024

Choose a reason for hiding this comment

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

Reverting this change as I didn't intentionally make it. If it was due to the updated fact that this vulnerability was introduced in v2, please let me know.

Update: Turns out I don't have the permission. I assume this change is required as well.

@joshmgross
Copy link
Copy Markdown

👋 Hey @JLHwung,

I can confirm that the vulnerable dependency was only introduced in @actions/artifact v2.0.1 - actions/toolkit#1602

Which means @actions/artifact versions before v2 and the actions/download-artifact v3 and below are not vulnerable

https://github.com/actions/download-artifact/blob/v3/package.json#L31

@advisory-database advisory-database bot merged commit fb1c3f3 into JLHwung/advisory-improvement-4752 Sep 4, 2024
@advisory-database
Copy link
Copy Markdown
Contributor

Hi @JLHwung! 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!

@advisory-database advisory-database bot deleted the JLHwung-GHSA-6q32-hq47-5qq3 branch September 4, 2024 14:00
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