Re: smtpserver router subdaemon

[Matti Aarnio <mea@nic.funet.fi>]

On Tue, Apr 13, 2004 at 04:08:37PM +0400, Eugene Crosser wrote:
> On Tue, 2004-04-13 at 15:49, Eugene Crosser wrote:
> 
> > smtpdata.o(.text+0x1468): In function `smtp_bdata':
> > /usr/local/src/zmailer/zmailer/smtpserver/smtpdata.c:574: undefined reference to `fd_statfs'
> > 
> > and more of similar kind.
> 
> Forget it, sorry. (after some hassle with autoconf, I apparently forgot
> to run "make clean" *again*)..

I suspected so :-)

> Unrelated, there still seem to be a couple missing diffs:
> 
> Index: smtpserver/policytest.c
> ===================================================================
> RCS file: /cvsroot/zmailer/smtpserver/policytest.c,v
> retrieving revision 1.89
> diff -u -r1.89 policytest.c
> --- smtpserver/policytest.c     12 Apr 2004 12:13:11 -0000      1.89
> +++ smtpserver/policytest.c     13 Apr 2004 10:19:58 -0000
> @@ -1704,7 +1704,9 @@      if (state->always_freeze)
>         return 1;
>      if (state->full_trust || state->authuser)
> -      return 0;
> +       return 0;
> +    if (state->always_accept)
> +       return 0;
>      if (len > 0) { /* Non-box address.. */

The story here is somewhat more complicated.
For the "ratelimitmsgs" to work, the "always_accept" flag testing
must be moved onwards - a lot.

Reading the code again, the "always_accept" has been demoted into
"accept if various tests don't reject" state, while "full_trust" is
literally that.

What I say above is true for MAIL FROM policy processing.
The RCPT TO policy processing needs similar forward movement.
 .. which is there already
just unplugging the unnecessarily early test.

> and
> 
> Index: proto/cf/aliases.cf
> ===================================================================
> RCS file: /cvsroot/zmailer/proto/cf/aliases.cf,v
> retrieving revision 1.55
> diff -u -r1.55 aliases.cf
> --- proto/cf/aliases.cf 12 Apr 2004 12:13:10 -0000      1.55
> +++ proto/cf/aliases.cf 13 Apr 2004 10:19:53 -0000
> @@ -228,7 +228,7 @@         user=$(user $quad)
> 
>         if [ "$DOMAIN_AWARE_GETPWNAM" = "1" ] ; then
> -               sysuser="$host@domain"
> +               sysuser="$user$domain"
>         else
>                 sysuser="$user"         fi
> 

Damn..  Ok, now it is fixed in source file.

-- 
/Matti Aarnio	<mea@nic.funet.fi>
-
To unsubscribe from this list: send the line "unsubscribe zmailer" in
the body of a message to majordomo@nic.funet.fi