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

crash-81154a65438ba5aaeca73fd502fa4850fbde60f8.tif triggers some Antiviruses #113

Closed
1 task done
HeavyTony2 opened this issue Feb 24, 2022 · 8 comments
Closed
1 task done

Comments

@HeavyTony2
Copy link

HeavyTony2 commented Feb 24, 2022

Issue: crash-81154a65438ba5aaeca73fd502fa4850fbde60f8.tif is considered as harfmul by some antivirus. See https://www.virustotal.com/gui/file/13b76234bb194c93a9a94653b4dd850ba5498f66f5f64e48e4109e7120cedc71?nocache=1


Environment (conda list):
$ conda list


Details about conda and system ( conda info ):
$ conda info

@HeavyTony2
Copy link
Author

Nevermind, I just saw this #104 (comment)

@johnbradley
Copy link
Contributor

johnbradley commented Mar 11, 2022

I am seeing this problem when I install pillow using conda. The crash-81154a65438ba5aaeca73fd502fa4850fbde60f8.tif file was still downloaded and picked up by some antivirus software. For example the bad crash file was downloaded to ~/.conda/pkgs/pillow-9.0.0-py310h2f95282_0/info/recipe/Tests/images/crash-81154a65438ba5aaeca73fd502fa4850fbde60f8.tif
Would it be possible to exclude this file from the conda package?

@h-vetinari
Copy link
Member

Can you raise a PR to remove the affected images and skip the respective tests?

@hugovk
Copy link
Contributor

hugovk commented Mar 14, 2022

@johnbradley Which antivirus software?

The file was already moved out of the main Pillow repo due to an AV false positive:

Feel free to do so here too.

And the test is already skipped in code when the file is not found:

Please could you also report the file to the AV software as a false positive? That would help others. It's a test file to guard against a decoding regression: python-pillow/Pillow#5372, not a virus file.

Thanks!

@h-vetinari
Copy link
Member

h-vetinari commented Apr 13, 2022

This image has been removed by #114.

@jakirkham
Copy link
Member

Sorry, but clarification is that meant to be has been?

@h-vetinari
Copy link
Member

Sorry, but clarification is that meant to be has been?

Yes! No idea why I mistyped this, sorry about the false alarm.

@jakirkham
Copy link
Member

All good. Thanks for clarifying 🙏

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

5 participants