You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@bugzilla.spamassassin.org on 2007/06/20 14:26:45 UTC
[Bug 5529] New: make test fail with t/spamd_plugin.t
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
Summary: make test fail with t/spamd_plugin.t
Product: Spamassassin
Version: SVN Trunk (Latest Devel Version)
Platform: Other
OS/Version: Linux
Status: NEW
Severity: critical
Priority: P2
Component: Building & Packaging
AssignedTo: dev@spamassassin.apache.org
ReportedBy: jacek_kal@o2.pl
First of all, it is not a bug #5416, #5510, #5525 duplicate!
SpamAssassin snapshot: spamassassin_20070620101200
During 'make test', test fails with:
t/spamd_plugin.................. Not found: called1 = test: called
myTestPlugin, round 1
# Failed test 2 in t/SATest.pm at line 648
Output can be examined in: log/d.spamd_plugin/out.1 log/d.spamd_plugin/spamd.err.1
t/spamd_plugin..................NOK 2/6 Not found: called2 = called
myTestPlugin, round 2
t/spamd_plugin..................NOK 4/6# Failed test 4 in t/SATest.pm at line
648 fail #2
Output can be examined in: log/d.spamd_plugin/out.1
log/d.spamd_plugin/spamd.err.1 log/d.spamd_plugin/out.3
log/d.spamd_plugin/spamd.err.1
Not found: called3 = called myTestPlugin, round 3
# Failed test 6 in t/SATest.pm at line 648 fail #3
Output can be examined in: log/d.spamd_plugin/out.1
log/d.spamd_plugin/spamd.err.1 log/d.spamd_plugin/out.3
log/d.spamd_plugin/spamd.err.1 log/d.spamd_plugin/out.5
log/d.spamd_plugin/spamd.err.1
t/spamd_plugin..................FAILED tests 2, 4, 6
Failed 3/6 tests, 50.00% okay
...
Failed Test Stat Wstat Total Fail List of Failed
-------------------------------------------------------------------------------
t/spamd_plugin.t 6 3 2 4 6
32 tests skipped.
Failed 1/138 test scripts. 3/1925 subtests failed.
Files=138, Tests=1925, 651 wallclock secs (303.72 cusr + 15.23 csys = 318.95 CPU)
Failed 1/138 test programs. 3/1925 subtests failed.
make: *** [test_dynamic] Error 255
If you want additional debug messages, please tell me.
The same error I got on 2 machines.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From spamassassin@dostech.ca 2007-07-23 01:38 -------
+1
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From jacek_kal@o2.pl 2007-06-20 23:00 -------
Created an attachment (id=4005)
--> (http://issues.apache.org/SpamAssassin/attachment.cgi?id=4005&action=view)
logs from t/log/d.spamd_plugin/
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
sidney@sidney.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |REOPENED
Resolution|FIXED |
Status Whiteboard| |needs 2 votes for 3.1 branch
Target Milestone|3.2.2 |3.1.10
------- Additional Comments From sidney@sidney.com 2007-07-17 05:50 -------
Reopening to target to 3.1.10
Here's my +1 vote to apply this fix to the 3.1 branch
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From jacek_kal@o2.pl 2007-06-24 03:38 -------
Sidney, sorry but you are just kidding that my /root is to much secured and I
should relax permissions (or I misunderstood your answer)? I'm an administrator
of several servers, and all of them have chmod 700 or 750 (where group is root).
It is a default system installation, and not my security. And nobody except root
doesn't have access to this directory.
And second part:
If I have to build spamassassin outside of /root directory, so make sure that
CPAN shell which is ran as root will extract package somewhere else than
~/.cpan/build (/root/.cpan/build) - which is default, where nobody user will
have execute access permissions.
One part of installation can be done from sources (and you are 100% right then I
can manually build package somewhere else). But most users are using CPAN to
install/update theirs system.
I don't know how this problem should be fixed from application (programming)
point of view. I'm not a specialist in this.
I have checked (with your help) where is a problem. I also used 'strace' to find
it out:
[pid 232] open("/root/src/spamassassin/t/log/spamd_plugin.tmp",
O_RDONLY|O_LARGEFILE) = -1 EACCES (Permission denied)
[pid 232] write(2, "[232] warn: Warning: something\'s wrong at
/root/src/spamassassin/t/log/localrules.tmp/../../data/testplugin.pm line 49,
<GEN7> line 83.\n", 136) = 136
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From jacek_kal@o2.pl 2007-06-21 08:50 -------
As I wrote in the first message, I have checked this patch. It is already
included in tested (SVN) version and the problem still occur.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From jm@jmason.org 2007-06-21 05:55 -------
actually I think this may be bug 5510.
please try applying the patch from
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5510#c7
and re-test...
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From sidney@sidney.com 2007-06-25 03:13 -------
> in the spamd t scripts which have this problem, we simply skip
> the entire t script if that method returns true.
I had just about come to that conclusion. I only have one test left that I
haven't got working, spamc_optC.t which uses the existence of a file in log/ to
signal a test plugin that it should fail, but it will be a heck of a lot cleaner
to first set the plugins that need to be changed to not be run, and then if we
want to, update them one by one.
I also came to the same conclusion about setting umask in SATest.pm instead of
having to track down every place in the tests that we need a chmod after
creating a file or directory.
I'm almost ready with a patch for all that.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
spamassassin@dostech.ca changed:
What |Removed |Added
----------------------------------------------------------------------------
Status Whiteboard|needs 1 vote |go
------- Additional Comments From spamassassin@dostech.ca 2007-07-04 12:05 -------
+1
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
jacek_kal@o2.pl changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|normal |critical
Priority|P4 |P2
------- Additional Comments From jacek_kal@o2.pl 2007-06-22 14:40 -------
Sorry, this fix also didn't solve a problem, but.... I've found a problem!
# ls -ld /root
drwx------ 20 root root 4096 2007-06-20 11:33 /root/
When perl will try to sudo nobody and try to access:
/root/.../spamassassin/t/log/spamd_plugin.tmp it will always fail.
The solution (I think) should be creating temporary files in /tmp (where all
users have write access).
If you would like to check this, try:
chmod 700 /root
and retry to run tests ;)
So currently I'm waiting for a working patch :) (after changing temporary chmod
/root to 701, all tests passed.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
jacek_kal@o2.pl changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|major |critical
------- Additional Comments From jacek_kal@o2.pl 2007-06-26 09:32 -------
+1 - also works fine for me.
And I think that problem was already fixed.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
sidney@sidney.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|[review] make test fail with|make test fail with
|t/spamd_plugin.t |t/spamd_plugin.t
Status Whiteboard|needs 1 vote |
------- Additional Comments From sidney@sidney.com 2007-06-24 04:23 -------
Jacek, sorry if I wasn't clear, but yes you did misunderstand my answer. I said,
"If you have set up your system so that /root is that secured, then do your
build in a directory you create somewhere else." I said "that secured" not "too
secured". I was suggesting that if you do want /root to be that secured then the
workaround could be to build in another directory.
But you are correct that my suggested workaround will not allow installation
from CPAN. That means that 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.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From sidney@sidney.com 2007-06-24 19:20 -------
Wow, this opened a can of worms... Testing the way Justin reproduced the problem
by setting umask 077 reveals a number of places where we use mkdir(foo, 0755)
which does not do the right thing because of the umask unless it is followed by
chmod(0755, foo).
But I think I'll have a patch soon.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
sidney@sidney.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status Whiteboard|needs 2 votes |needs 1 vote
------- Additional Comments From sidney@sidney.com 2007-06-24 02:27 -------
+1 on Justin's fix. I think it is necessary even if we do something else to deal
with the problem of root's home directory being inaccessible.
Checking in that fix may or may not be sufficient for closing this bug. That
would be a judgment call that I'm not making right now.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From jacek_kal@o2.pl 2007-06-22 02:17 -------
There could be some true. I have tried to compile and run tests as a normal
(unprivileged user), and all tests passed.
I'll attach logs from a normal user.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From jm@jmason.org 2007-06-22 09:09 -------
Created an attachment (id=4015)
--> (http://issues.apache.org/SpamAssassin/attachment.cgi?id=4015&action=view)
fix
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From jm@jmason.org 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.
[Bug 5529] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From jm@jmason.org 2007-06-20 05:42 -------
the contents of t/log/d.spamc_plugin would be helpful. thanks!
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
sidney@sidney.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
Status Whiteboard|go |
------- Additional Comments From sidney@sidney.com 2007-07-04 12:35 -------
Committed to branch 3.2 revision 553310.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
jm@jmason.org changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|critical |major
Summary|make test fail with |[review] make test fail with
|t/spamd_plugin.t |t/spamd_plugin.t
Status Whiteboard| |needs 1 vote
------- Additional Comments From jm@jmason.org 2007-06-26 07:33 -------
+1 -- that approach is fine by me, also all tests now pass when run with:
sudo bash
chmod 700 .
chown root .
umask 077
rm -rf t/log
make test
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
sidney@sidney.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #4015 is|0 |1
obsolete| |
------- Additional Comments From sidney@sidney.com 2007-06-25 22:45 -------
Created an attachment (id=4023)
--> (http://issues.apache.org/SpamAssassin/attachment.cgi?id=4023&action=view)
Simple patch that should allow CPAN builds to work
This patch seems to be the minimum that would allow a CPAN install to work by
letting make test using the default settings in t/config.dist to run without
failures even if running as root in a directory that has no permissions for
non-root.
It sets umask to 022 if SATest.pm is run as root, it makes sure that t/log has
world read/execute permissions even if it exists already, and it creates a new
flag, $SKIP_SETUID_NOBODY_TESTS, that tests can use to skip execution if
running as root in a directory that non-root can't access.
For this patch, that last piece is still incomplete in that I only test for
running as root. Eventually I would like to actually setuid nobody and test
access to . and/or ./t/log and only conditionally set the flag. I also want to
create a tmp directory under those circumstances. Doing all that should allow
t/spamd_plugin.t to run under root. For now, this patch just disables that test
when running as root, which I think is a reasonable tradeoff.
Note that this patch does not allow the t/root*.t tests to be run in a
directory that does not provide world read/execute access. I don't disable the
tests when $SKIP_SETUID_NOBODY_TESTS is set because for now that would mean
never running them, and by default config.dist disables them.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From jacek_kal@o2.pl 2007-06-22 14:46 -------
And once more - how it is possible that all users have chmod 755 for /root
directory (if no one can duplicate this bug)? It is quite strange for me (from
security reason) ;)
But maybe I've applied paranoid security to all of my systems :D
And once more thank you for your help - without it, I will never found this.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
jm@jmason.org changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|critical |normal
Priority|P2 |P4
Summary|make test fail with |[review] make test fail with
|t/spamd_plugin.t |t/spamd_plugin.t
Status Whiteboard| |needs 2 votes
------- Additional Comments From jm@jmason.org 2007-06-22 09:10 -------
not checked into trunk yet; there are conflicting changes from other bugs in the
3.2.2 queue :(
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
jm@jmason.org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status Whiteboard|needs 2 votes for 3.1 branch|needs 1 vote for 3.1 branch
------- Additional Comments From jm@jmason.org 2007-07-18 06:36 -------
+1
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From sidney@sidney.com 2007-06-26 13:25 -------
Committed to trunk revision 550927
> I think that problem was already fixed
In comment #10 you said that it wasn't, and I confirmed that simply changing the
permissions of log would not do it if /root was still 0700. The problem is that
testplugin.pm read/writes to a file to use it as a counter and the plugin runs
as nobody. I had to add something to spamd_plugin.t to skip the test under these
circumstances. Eventually I would like to have SATest.pm set up a temp directory
under /tmp that tests can use if they need to have setuid nobody spamd processes
access files, but that was more extensive changes than I wanted just to allow
CPAN installs.
Note that this still does not allow running the t/root* tests unless you build
SpamAssassin in a directory that is world-readable, and it does skip the
spamd_plugin.t test if you are running as root. So to fully test SpamAssassin
with this patch you would, for example, copy it as non-root in some directory
that is readable by nobody, make and make test, then sudo make test.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
jm@jmason.org changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|Undefined |3.2.2
------- Additional Comments From jm@jmason.org 2007-06-22 09:08 -------
it's a perms issue, looking at these lines in d.spamd_plugin/spamd.err.1 :
[1417] warn: Warning: something's wrong at
/root/src/spamassassin/t/log/localrules.tm
p/../../data/testplugin.pm line 49, <GEN7> line 83.
[1417] warn: readline() on closed filehandle IN at
/root/src/spamassassin/t/log/local
rules.tmp/../../data/testplugin.pm line 50.
[1417] dbg: test: called myTestPlugin, round 0
[1417] warn: Warning: something's wrong at
/root/src/spamassassin/t/log/localrules.tm
p/../../data/testplugin.pm line 55.
[1417] warn: print() on closed filehandle OUT at
/root/src/spamassassin/t/log/localru
les.tmp/../../data/testplugin.pm line 56.
[1417] warn: Warning: something's wrong at
/root/src/spamassassin/t/log/localrules.tm
p/../../data/testplugin.pm line 57.
I was able to repro it by doing:
sudo bash
umask 077
cd t
rm -rf log
./spamd_plugin.t
this patch fixes it for me.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From jacek_kal@o2.pl 2007-06-22 02:18 -------
Created an attachment (id=4009)
--> (http://issues.apache.org/SpamAssassin/attachment.cgi?id=4009&action=view)
logs from t/log/d.spamd_plugin for unpriv user
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
spamassassin@dostech.ca changed:
What |Removed |Added
----------------------------------------------------------------------------
Status Whiteboard|needs 1 vote for 3.1 branch |go
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
------- Additional Comments From sidney@sidney.com 2007-06-24 00:06 -------
I think that Justin's patch fixes a problem that is a reasonable configuration
-- root has umask set to something less permissive than 0755 -- so it is worth
doing. But I don't think that we need to go to the extreme of putting all the
log output in a tmp directory to take care of running the install and make test
as root. Instead, the installation instructions could specify that it must be
built in a directory that provides general read access. If you have set up your
system so that /root is that secured, then do your build in a directory you
create somewhere else.
Perhaps we could add a test in SATest.pm that the parents of the current
directory provide at least execute permission? Or test that if running as root
it is possible to suid nobody and read the log directory?
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5529] [review] make test fail with t/spamd_plugin.t
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5529
sidney@sidney.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|REOPENED |RESOLVED
Resolution| |FIXED
------- Additional Comments From sidney@sidney.com 2007-07-23 02:55 -------
Committed to branch 3.1 revision 558677.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.