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-w687-f44x-x42j false positive? #419

Closed
cgauld opened this issue Jun 20, 2022 · 7 comments
Closed

GHSA-w687-f44x-x42j false positive? #419

cgauld opened this issue Jun 20, 2022 · 7 comments

Comments

@cgauld
Copy link

cgauld commented Jun 20, 2022

We received a dependabot alert about this advisory:
GHSA-w687-f44x-x42j

It's very strange, because this isn't an NPM package - it's a built-in element of the Unity game engine and is referenced from their "package.json" manifest files (not related to NPM).

Are there any more details about this advisory and if it is indeed an issue with Unity packages?

Thanks for any insight!

@ViMaSter
Copy link

ViMaSter commented Jun 20, 2022

I suppose this might be related: https://www.npmjs.com/package/com.unity.modules.audio
image

If I had to hazard a guess, someone uploaded an infected package to npm with the exact same name of the package from upm (Unity Package Manager) in an attempt to accidentally have someone download from npm rather than upm. Since Unity borrowed most of the syntax (including package.json), I guess there's no way for GitHub to know whether the npm or upm package is actually referenced.

However, I agree; additional clarification or confirmation on this would be greatly appreciated.


Edit: This seems to not be limited to just one unity-related advisory:

@LegoCylon
Copy link

LegoCylon commented Jun 20, 2022

Post processing got flagged, too. GHSA-3x2r-7cgg-82q2 (com.unity.postprocessing)

@cgtinker
Copy link

cgtinker commented Jun 20, 2022

com.unity.modules.physics GHSA-pm97-j6q8-376p got flagged swell

Is any action required when everything has been loaded via unity hub / unity package manager?
...Just read some articles and there have been npm malware injections in the past (within unity). This is concerning, it would be nice to know either how long the malware npm was downloadable or how to find out if the system is compromised.

@brunohstein
Copy link

Guess it's a similar issue here? GHSA-fw55-8gwc-gf65

Captura de Tela 2022-06-20 às 19 56 03

Have the first one installed but got a dependabot alert about the second one.

@jordan-zilch
Copy link

Do Unity packages ever auto-update? It looks like the malicious com.unity.textmeshpro (GHSA-cvgg-5542-9692) was only published 10 months ago, so I'm assuming that I'm safe since I haven't opened my Unity project file since then.

@darakian
Copy link
Contributor

darakian commented Oct 18, 2022

Apologies for the late reply on this thread, but for clarity what's happening is that malware is being uploaded to npm. These advisories are for the packages on npm and not for the unity packages. We have a known issue with dependabot where unity package.json files are being interpreted as npm files. I can't comment on a timeline, but we're aware of the issue and working toward a fix.

We suspect the malware is being uploaded to npm to squat the unity package names in the hopes that users also mix up npm and unity package.json files.

See also: #516

@darakian
Copy link
Contributor

Hey all 👋

Quick update on this. I'm gonna close this issue out as we have just recently merged in a fix for this issue. If you see more erroneous alerts please feel free to reopen this issue or to make a new one 👍

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

No branches or pull requests

7 participants