First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 136445
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Qmail Team <qmail-bugs@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Jakub Moc <jakub@gentoo.org>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 136445 depends on: Show dependency tree
Show dependency graph
Bug 136445 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)







View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2006-06-11 10:41 0000
RESTRICT="usersandbox" is not honored by portage... not really sure what you
are after here.

------- Comment #1 From Robin Johnson 2006-06-11 13:37:13 0000 -------
IIRC, the packages fail to compile if you have FEATURES=usersandbox.
userpriv on it's own is fine, and sandbox on it's own is fine.

------- Comment #2 From Jakub Moc 2006-06-11 13:47:36 0000 -------
(In reply to comment #1)
> IIRC, the packages fail to compile if you have FEATURES=usersandbox.
> userpriv on it's own is fine, and sandbox on it's own is fine.

Well, it doesn't do anything useful, portage ignores it. :) According to
changelog, this has been added for Bug 93447 in qmail-scanner.
RESTRICT="userpriv" is the effective restrict there, that usersandbox thing is
just redundant and silently ignored.

As for qmail, this has been added for Bug 99390. It seems kinda redundant to
me, the bug was about tests, those are restricted, so there's no need for
additional restrictions there, or does it fail w/ userpriv as well without
tests enabled? 

Couldn't find anything useful wrt these restricts in changelog for netqmail.

------- Comment #3 From Michael Hanselmann (hansmi) (RETIRED) 2006-06-17 04:33:56 0000 -------
Fixed in CVS, thanks.

------- Comment #4 From Jakub Moc 2006-06-19 15:21:45 0000 -------
Still there in qmail-scanner ebuilds... ;)

------- Comment #5 From Jakub Moc 2006-07-21 02:29:13 0000 -------
 20 Jul 2006; Michael Sterrett <mr_bones_@gentoo.org>
  qmail-scanner-1.25-r1.ebuild, qmail-scanner-2.01.ebuild:
  usersandbox => sandbox since usersandbox isn't a valid RESTRICT

------- Comment #6 From Robin Johnson 2006-07-21 02:53:29 0000 -------
Reopening this.
Usersandbox needs to become a valid RESTRICT.

qmail-scanner is a special case.
The 'configure' script has always been heavily hacked by upstream, and has in
the past done unsafe things - mainly around writing to the stuff used by an
active install of qmail-scanner, and breaking the installed copy by doing so
(and losing mail). 

With FEATURES='userpriv' OR FEATURES='sandbox', it compiles and works fine (and
the compile is safe).
FEATURES='userpriv sandbox' is the same as FEATURES='userpriv' (sandbox is not
used).
With FEATURES='-sandbox', it compiles, but you're vulnerable to upstream
stupidity.
With FEATURES='userpriv usersandbox', it fails to compile and work properly.

So specifically, we want to ONLY block usersandbox.

Mr_bones_: would you please revert your "usersandbox => sandbox" change ASAP to
avoid qmail-scanner breaking existing systems.

------- Comment #7 From Jakub Moc 2006-09-23 11:34:43 0000 -------
*** Bug 148815 has been marked as a duplicate of this bug. ***

------- Comment #8 From Konstantin 2007-06-14 11:47:25 0000 -------
I remembered, that I've workarounded this problem downgrading clamav, installed
qmail-scanner and after that upgrading clamav. But with latest clamav versions
you cannot reemerge mail-filter/qmail-scanner.

------- Comment #9 From Jakub Moc 2007-12-13 22:54:32 0000 -------
# grep RESTRICT *.ebuild
qmail-scanner-1.25-r1.ebuild:RESTRICT="userpriv"
qmail-scanner-2.01.ebuild:RESTRICT="userpriv"

(In reply to comment #6)
> Reopening this.
> Usersandbox needs to become a valid RESTRICT.

Why? RESTRICT="usersandbox" will yield you either effective FEATURES='userpriv
sandbox' (which translates to running as portage:portage without a sandbox) or
effective FEATURES='sandbox' (sandboxed with root privs). What's the exact
expected benefit from making usersandbox a valid restrict I really fail to see,
you already get FEATURES='sandbox' with the current RESTRICT="userpriv".

Closing this as the RESTRICT is fixed.

First Last Prev Next    No search results available      Search page      Enter new bug