On Thu, Mar 13, 2008 at 09:17:34AM +0000, Marcin Owsiany wrote:
> I may have a closer look today or during the weekend, but it may make more
> sense for someone who knows the API better to jump in at this point.

I had a look, and it looks like not resetting the recursion level is libspf's
fault, as:
 - there is already a place for it to do it in (somewhere in SPF_smtp_from())
 - I cannot think of any reason not to do it there (keeping the current
   recursion level when switching sender does not make much sense)
 - from encapsulation POV it's better to do it in the library rather than have
   the library client (spfmilter in this case) fiddle with members of peer_info
   structure.

I have just filed a bug against libspf #470758 asking whether it is
appropriate to resolve it there.

There is also a tested patch against libspf0 in that bug.

-- 
Marcin Owsiany <[EMAIL PROTECTED]>             http://marcin.owsiany.pl/
GnuPG: 1024D/60F41216  FE67 DA2D 0ACA FC5E 3F75  D6F6 3A0D 8AA0 60F4 1216



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to