9 messages in net.sourceforge.lists.courier-usersRe: [courier-users] [PATCH] SPF
FromSent OnAttachments
Mark ConstableJan 24, 2008 7:48 pm 
Alessandro VeselyJan 24, 2008 10:28 pm 
Alessandro VeselyJan 24, 2008 10:46 pm 
Mark ConstableJan 24, 2008 11:17 pm 
Alessandro VeselyJan 25, 2008 2:17 am 
Mark ConstableJan 25, 2008 2:56 am 
Alessandro VeselyJan 26, 2008 7:28 am 
Mark ConstableJan 28, 2008 6:35 pm 
Sam VarshavchikJan 28, 2008 7:54 pm 
Actions with this message:
Paste this link in email or IM:
Paste this link in email or IM:
Atom feed for this thread
Paste this URL into your reader:
Subject:Re: [courier-users] [PATCH] SPFActions...
From:Alessandro Vesely (ves@tana.it)
Date:Jan 26, 2008 7:28:44 am
List:net.sourceforge.lists.courier-users

Mark Constable wrote:

On 25 Jan 2008 16:29, Alessandro Vesely wrote:

That's very informative, excellent, thank you.

Hm... looking closer it may be wrong or different than what I thought it was. In facts, testing it at mine I get this:

wmail-505# cd .. wmail-506# cd rfc1035 wmail-507# ./testspf -h Usage: ./testspf mailfrom remoteip remotehost helo me wmail-508# wmail-508# ./testspf "post@bankwest.com.au" "216.82.241.83" "" "" "" fail: Address does not pass the Sender Policy Framework wmail-509# wmail-509# wmail-509# !! ./testspf "post@bankwest.com.au" "216.82.241.83" "" "" "" fail: Address does not pass the Sender Policy Framework wmail-510# ./testspf "post@bankwest.com.au" "::ffff:216.82.241.83" "" "" "" fail: Address does not pass the Sender Policy Framework

What's wrong? The program queries "nets.messagelabs.com", i.e. the "%{ir}" macro results in a null string expansion.

After tinkering a bit I applied the attached patch and the same command line above this time gave a "pass". You may have the same problem, if it hasn't been already fixed.

--

--- spf.original.c 2006-04-13 01:00:49.000000000 +0200 +++ spf.c 2008-01-26 16:21:19.000000000 +0100 @@ -1390,7 +1390,7 @@ ** number, but MUST support at least a value of 9. */

- if (transformer_count > n) + if (transformer_count > n || transformer_count <= 0) transformer_count=n;

if (transformer_reverse)