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

🐛 Bug Report: ClamAv::fileScanInStream does not detect eicar.com file #26

Open
2 tasks done
vagiguere opened this issue May 24, 2023 · 2 comments · May be fixed by #27
Open
2 tasks done

🐛 Bug Report: ClamAv::fileScanInStream does not detect eicar.com file #26

vagiguere opened this issue May 24, 2023 · 2 comments · May be fixed by #27
Labels
bug Something isn't working

Comments

@vagiguere
Copy link

👟 Reproduction steps

Download the "eicar.com" or the "eicar.com.txt" file from https://www.eicar.org/download-anti-malware-testfile/

$clam = new Network($host, $port);
$is_clean = $clam->fileScanInStream('path/to/eicar/file');

The function is supposed to return false, instead it returns true.
This behavior happen with both "eicar.com" and "eicar.com.txt" files but not with zip files.

The problem seem to happen with text files of less than 8192 bytes.

In Appwrite\ClamAv\ClamAv:107 the fread return an empty string which is sent to the socket. The problem is ClamAv does not detect the eicar file if this empty string is packed and send through the socket.

👍 Expected behavior

in the example above the $is_clean variable should be false.

👎 Actual Behavior

In the exemple above the $is_clean variable is true.

🎲 PHP ClamAV version

Different version (specify in environment)

💻 Operating system

Linux

🧱 Your Environment

Version : appwrite/php-clamav:2.0.0
PHP 8.2 (also with 8.1) in a docker container
Clamav in another docker container : https://hub.docker.com/r/clamav/clamav/

👀 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

@pehbehbeh
Copy link

Same problem here. Have you been able to find a solution?

@vagiguere
Copy link
Author

Yes, the PR here (#27).
I'm gonna test the changes from #27 (comment) and hope the appwrite team will accept the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants