diff options
author | Sunil Shetye <shetye@bombay.retortsoft.com> | 2010-10-14 00:46:31 +0530 |
---|---|---|
committer | Matthias Andree <matthias.andree@gmx.de> | 2010-10-21 12:17:13 +0100 |
commit | 361cefb5747b93aa3a6510af017ec89390673c38 (patch) | |
tree | c125ffd35c0a140619997692563fe2dcb3bdc5ff /sink.c | |
parent | fa2ac589a4dccb25e9da1757f53e2b25d1d3e1a3 (diff) | |
download | fetchmail-361cefb5747b93aa3a6510af017ec89390673c38.tar.gz fetchmail-361cefb5747b93aa3a6510af017ec89390673c38.tar.bz2 fetchmail-361cefb5747b93aa3a6510af017ec89390673c38.zip |
Fix "antispam does not work"
This appears to be similar to the recent --nosoftbounce and --nobounce
issue where command line options are not handled exactly the same way as
the rcfile parser handles them.
Reported by Kees Bakker, BerliOS bug #17599.
Diffstat (limited to 'sink.c')
-rw-r--r-- | sink.c | 12 |
1 files changed, 1 insertions, 11 deletions
@@ -441,8 +441,6 @@ static int handle_smtp_report(struct query *ctl, struct msgblk *msg) { int smtperr = atoi(smtp_response); char *responses[1]; - struct idlist *walk; - int found = 0; responses[0] = xstrdup(smtp_response); @@ -466,15 +464,7 @@ static int handle_smtp_report(struct query *ctl, struct msgblk *msg) * messages, which are probably in English (none of the * MTAs I know about are internationalized). */ - for( walk = ctl->antispam; walk; walk = walk->next ) - if ( walk->val.status.num == smtperr ) - { - found=1; - break; - } - - /* if (str_find(&ctl->antispam, smtperr)) */ - if ( found ) + if (str_find(&ctl->antispam, smtperr)) { /* * SMTP listener explicitly refuses to deliver mail |