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

Windows defender flags ninja.exe as containing a trojan #33

Closed
mtreinish opened this issue Jun 11, 2020 · 8 comments
Closed

Windows defender flags ninja.exe as containing a trojan #33

mtreinish opened this issue Jun 11, 2020 · 8 comments
Labels
Status: In Progress This item is in progress.

Comments

@mtreinish
Copy link

Pip installing ninja (via a setup dependency for a project relying on scikit-build to build) is failing because windows defender is blocking ninja.exe. It is flagging ninja.exe in the wheel as containing a trojan:

Annotation 2020-06-11 130816

@jcfr
Copy link
Contributor

jcfr commented Jun 11, 2020

Thanks for bringing this up 🙏

Let's try to have a look:

Is defender raising an alarm after downloading https://github.com/kitware/ninja/archive/v1.10.0.gfb670.kitware.jobserver-1.zip ?

Could you also check if definer complain after downloading the binary provided by the upstream project ? See https://github.com/ninja-build/ninja/releases/download/v1.10.0/ninja-win.zip

Thanks for your help,

@jcfr jcfr added the Status: Awaiting response ⏳ Waiting for a response/more information label Jun 11, 2020
@mtreinish
Copy link
Author

I download both https://github.com/kitware/ninja/archive/v1.10.0.gfb670.kitware.jobserver-1.zip and https://github.com/ninja-build/ninja/releases/download/v1.10.0/ninja-win.zip on my windows 10 vm, extracted both and also manually scanned both of the extracted zips. Nothing was flagged by windows defender. Although for https://github.com/kitware/ninja/archive/v1.10.0.gfb670.kitware.jobserver-1.zip I didn't see a binary in the extracted contents.

But, I did also download: https://github.com/Kitware/ninja/releases/download/v1.10.0.gfb670.kitware.jobserver-1/ninja-1.10.0.gfb670.kitware.jobserver-1_i686-pc-windows-msvc.zip from the release page on the Kitware/ninja repo and that was flagged. So it looks like the source of the infected binary is that

@mtreinish
Copy link
Author

We probably should report this to the Kitware/ninja maintainers and pull the infected wheels from pypi in the meantime

@jcfr
Copy link
Contributor

jcfr commented Jun 11, 2020

Nothing was flagged by windows defender. Although for https://github.com/kitware/ninja/archive/v1.10.0.gfb670.kitware.jobserver-1.zip I didn't see a binary in the extracted contents.

Good point, I referenced the wrong link. I edited my comment with the correct link that you also looked at.

Kitware/ninja maintainers and pull the infected wheels from pypi in the meantime

Good news is that I work at Kitware, I will engage with the relevant team and report back.

https://github.com/ninja-build/ninja/releases/download/v1.10.0/ninja-win.zip on my windows 10 vm, extracted both and also manually scanned both of the extracted zips. Nothing was flagged by windows defender

Thanks for checking.

@jcfr jcfr added Status: In Progress This item is in progress. and removed Status: Awaiting response ⏳ Waiting for a response/more information labels Jun 11, 2020
@jcfr
Copy link
Contributor

jcfr commented Jun 11, 2020

Waiting this is sorted out, I just deleted the windows wheel from the release.

@jcfr
Copy link
Contributor

jcfr commented Jun 11, 2020

image

johnor added a commit to johnor/n_e_s that referenced this issue Jun 14, 2020
* Looks like the Windows wheels were removed from pypi:
  scikit-build/ninja-python-distributions#33
johnor added a commit to johnor/n_e_s that referenced this issue Jun 15, 2020
* Looks like the Windows wheels were removed from pypi:
  scikit-build/ninja-python-distributions#33
@jcfr
Copy link
Contributor

jcfr commented Jun 15, 2020

Analyzing the executable with VirusTotal didn't report any problem
https://www.virustotal.com/gui/file/ff1abc0a47838ed45f0abd68c6afed2d647edf3b225959dfd5ba6725b8106719/detection

Running it through Microsoft Safety Scanner as well as ESET did not detect anything.

@mtreinish

We would recommend submitting it to Microsoft as a false positive (Requires a Microsoft account)
https://www.microsoft.com/en-us/wdsi/filesubmission

Here you can get an overview of the process
https://docs.microsoft.com/en-us/windows/security/threat-protection/intelligence/submission-guide

@jcfr
Copy link
Contributor

jcfr commented Jun 15, 2020

We also updated https://github.com/Kitware/ninja/releases/download/v1.10.0.gfb670.kitware.jobserver-1/ninja-1.10.0.gfb670.kitware.jobserver-1_i686-pc-windows-msvc.zip to include a signed executable.

I will now generate 1.10.0.post1 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: In Progress This item is in progress.
Projects
None yet
Development

No branches or pull requests

2 participants