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!
>
>
Received on Tue Feb 5 07:44:20 2008
This archive was generated by hypermail 2.1.8 : Wed Feb 06 2008 - 00:00:12 CET