Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 708734 - acct-user/clamav clobbers existing supplementary groups
Summary: acct-user/clamav clobbers existing supplementary groups
Status: RESOLVED DUPLICATE of bug 708560
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Michael Orlitzky
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-02-08 16:11 UTC by Daniel M. Weeks
Modified: 2020-02-10 10:42 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel M. Weeks 2020-02-08 16:11:31 UTC
It is a common practice to use clamd and amavisd-new together running as different users. This requires putting clamav in the amavis group so it can read/scan files as they pass through amavis.

However, the installation of acct-user/clamav on a working system causes the supplementary groups of the clamav user to be rewritten, dropping access to amavis group files, breaking clamd's ability to perform a scan.

It seems this type of situation could affect other acct-user/* packages with great annoyance to users and potentially introduce security issues instead of solve them.

Reproducible: Always
Comment 1 Tomáš Mózes 2020-02-08 20:19:30 UTC

*** This bug has been marked as a duplicate of bug 708560 ***
Comment 2 Michael Orlitzky gentoo-dev 2020-02-08 22:38:17 UTC
I know that better than anyone...

But in this case, the recommended way to integrate clamav and amavisd-new was overly-permissive, and there's just a much better way to do it. The CONTSCAN method asks clamd to perform the scan itself, which means that it has to be able to read ALL of your personal files, not just the one being scanned. Running clamdscan with --fdpass is better in every way.

Now that amavis has an active upstream again, we can hopefully get the default entry replaced:

  https://gitlab.com/amavis/amavis/issues/59