Excellent! Thanks for some more eyes on this. I'll take a look over
the next couple days. I think what he is seeing is the underlying bug
that is causing NULL to be passed to the driver for the group name from
the CTX. The patch I provided fixes some of the logic to not fail
entirely; previously the getpwnam() call probably returned NULL so the
whole thing blew up. At least now you get some useful information.
Tony -- what is your retraining mail config that results in those args
passed to dspam?
-J
Mick Johnson wrote:
Jason Axley skrev, on 04-02-2008 08:21:
I have the attached patch to the mysql driver that should go in.
It's
part of the 3.8.1cvs build on debian/ubuntu and fixes some
problematic
bugs that break global groups that would be good to include. I don't
believe it has been included in the kirya.net build yet, but I've
been
running it on my busy server and it is like a dream...
On my FC6 (test) machine I'm using a shared group:
leerlingen:shared:*tru.leerlingen
I downloaded the CVS tree this morning and built it with this patch.
The
patch solved the problem I had with the CVS MySQL driver not being able
to find the addressee's uid in the DB, but assigns the wrong uid on
retraining:
input args: /usr/bin/dspam --class=spam --source=error --deliver=spam
--user leerlingen - it uses the addressee's uid (2) instead of the
group's uid (1). The original 3.8.0 driver is pretty horrible at
reporting not finding things (debug log), but it doesn't make that
mistake in particular.
I haven't been through the affected code as I'm no expert there, but it
seems to me that the patch is dealing pretty thoroughly with merged
groups while ignoring others.
Patch looks good - thanks Jason. However I'm holding off on committing it
until you can verify whether the issue Tony raised has been fixed (thanks
for testing it out Tony).
cheers
mick
!DSPAM:4,47a7a32e36672089823019!
