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

Malware Patrol problems #381

Open
justcsdr opened this issue Mar 26, 2021 · 4 comments
Open

Malware Patrol problems #381

justcsdr opened this issue Mar 26, 2021 · 4 comments
Milestone

Comments

@justcsdr
Copy link

May I suggest a patch for bugs in Malware Patrol?

First part is global. Is trimming the lines with more than 8189 characters because ClamAV doesn't like them. I have encountered it only in Malware Patrol db but it could happen to others too.

The second part is specific to Malware Patrol. It has the nasty habbit to blacklist the entire Google Drive path and some of our partners are using Google Drive.

You can implement it however fits you. These are just suggestions and I have implemented them in this for to keep the changes to your code at minimum.

The LC_ALL=en_US.UTF-16 part is for /var/lib/clamav-unofficial-sigs/dbs-is/interserver256.hdb

clamav-unofficial-sigs.sh.patch.txt

@extremeshok
Copy link
Owner

Please can you create a PR against the Dev branch. Then I can test and release it in the next version.

@perplexityjeff
Copy link
Contributor

Could this be made an option as well so this behavior can be turned on or off? As we are not dealing with these kind of partners but are getting a lot of Google Drive related viruses.

Thank you for all the hard work you all do.

@perplexityjeff
Copy link
Contributor

I created a pull request for the change, @justcsdr feel free to take a look as well, I based it on your code.

@justcsdr
Copy link
Author

There isn't much to say about it. To me it looks OK. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants