Re: 3.9.1-rc1: issue with classification user

Stevan Bajić <[email protected]>
Newsgroups gmane.mail.spam.dspam.devel
Message-ID <[email protected]>
On Mon, 09 Aug 2010 14:07:34 +0200
Tom Hendrikx <[email protected]> wrote:

> On 08/08/10 17:57, Stevan Bajić wrote:
> > On Sun, 08 Aug 2010 15:52:58 +0200
> > Tom Hendrikx <[email protected]> wrote:
> > 
> > [...]
> >>>
> >>> # cat /var/spool/dspam/group.away
> >>> groupname:classification:*classicifation_user
> >>>
> >>>> Could you post the output of "select * from dspam_virtual_uids where username = 'classification_user'"?
> >>>
> >>> vmail=# select * from dspam_virtual_uids;
> >>>  uid |        username
> >>> -----+------------------------
> >>>    1 | classification_user
> >>>    2 | [email protected]
> >>> (2 rows)
> >>>
> >>
> >> And a few minutes after sending the mail, I thought that I might have
> >> made a typo. And I did: classification_user != classicifation_user
> >>
> > Good catch.
> > 
> > 
> >> Sorry for the noise. However, I still think that an error message
> >> explaining the issue would help a lot.
> >>
> >> Error: processing message for user [email protected] failed: unable
> >> to find user 'classicifation_user' for peer classification.
> >>
> >> looks so much better than:
> >>
> >> bailing on error -2
> >> process_message returned error -5.  delivering.
> >>
> > You are right. The error message could be better. However... it's not that simple. I anyway need to change the whole code for better error handling in case of non existing users.
> > 
> > Could you try the attached patch? It will not print out the error message you requested but should work better in cases when ensure_confident_result() is returning a wrong result.
> > 
> > Could you send me a new debug output with the applied patch?
> 
> New debug output attached.
> 
> According to the debug file, a valid signature is generated, which is
> good. Unfortunately, it is not added to the database, and the message is
> not delivered to the user.
> 
> Postfix says in the DSN back to the sender:
> 
> <[email protected]> (expanded from <[email protected]>): Command died with
>     signal 11: "/usr/bin/dspam"
> 
Uhhh... a segmentation fault. Geee! That is in no way good!


> So this is actually a worse situation:)
> 
Yes! Worse then before.


> Regards,
> 	Tom
-- 
Kind Regards from Switzerland,

Stevan Bajić

------------------------------------------------------------------------------
This SF.net email is sponsored by 

Make an app they can't live without
Enter the BlackBerry Developer Challenge
http://p.sf.net/sfu/RIM-dev2dev 
_______________________________________________
Dspam-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dspam-devel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.