http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529





------- Additional Comments From [EMAIL PROTECTED]  2007-06-25 02:53 -------
(In reply to comment #15)
> we have to fix this in a way that lets the test work
> even inside a fully secured /root directory. So I agree with you.
> 
> I am now -1 on the patch because it does not fix the problem when /root has 
> 700
> or 750 permissions.
> 
> My guess is that it will be less work to fix this in those tests that use 
> setuid
> to non-root, rather than try to move the log directory to /tmp.

Here's an easier fix -- we come up with a method in SATest.pm which determines
if we're running as root, and one of the directories in the cwd is unreadable by
non-root, possibly by setuid'ing to non-root and testing.  Then, in the spamd t
scripts which have this problem, we simply skip the entire t script if that
method returns true.

We don't need to support the "running tests as root from inside an
unreadable-by-non-root directory" case, at all.  If those tests pass as
non-root, that's fine; and we have the t/root*.t scripts for when we need to
test something
explicitly as root.


Also: why don't we just reset the umask in t/SATest.pm to something like 022
before running any of the tests?



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Reply via email to