Oups yes, you sent me a diff too,
thanks ;)
On Mon, Apr 09, 2012 at 10:21:42PM +0200, Christopher Zimmermann wrote:
> Begin forwarded message:
>
> Hi,
>
> Yes this bug is still open because we've been focusing primarily on
> fixing parts of smtpd that could cause crashes. We will be solving bugs
> related to aliases shortly as Eric and I started discussing them just
> an hour ago.
>
> Feel free to join #opensmtpd @ freenode to participate ;)
>
> Gilles
>
> nb: jacekm no longer contributes to smtpd, in the future please cc:
> eric@ and chl@
>
>
> On Thu, Feb 02, 2012 at 02:48:21PM +0100, Christopher Zimmermann wrote:
> > Hi,
> >
> > I just noticed this bug is still outstanding. I have a patch attached
> > that fixes this problem for me, but I was told there was some clean up
> > to be done in lka_session.c before this can be fixed.
> >
> >
> > Christopher
> >
> >
> > Index: aliases.c
> > ===================================================================
> > RCS file: /cvs/src/usr.sbin/smtpd/aliases.c,v
> > retrieving revision 1.44
> > diff -u -p -r1.44 aliases.c
> > --- aliases.c 11 Oct 2011 17:57:10 -0000 1.44
> > +++ aliases.c 2 Feb 2012 13:48:03 -0000
> > @@ -87,6 +87,8 @@ aliases_get(objid_t mapid, struct expand
> > /* foreach node in map_alias expandtree, we merge */
> > nbaliases = 0;
> > RB_FOREACH(expnode, expandtree, &map_alias->expandtree) {
> > + (void)strlcpy(expnode->as_user, username,
> > + sizeof (expnode->as_user));
> > if (expnode->type == EXPAND_INCLUDE)
> > nbaliases +=
> > aliases_expand_include(expandtree, expnode->u.buffer); else {
> >
> >
> >
> > On Sun, 11 Dec 2011 22:28:45 +0100
> > Christopher Zimmermann <madro...@zakweb.de> wrote:
> >
> > > Hi,
> > >
> > > I want to use a pipe in my aliases, like this test case:
> > >
> > > madroach: |true
> > >
> > > but smtpd says the following. Look especially for the
> > > "forkmda: to true as" line. Seems like somewhere the username gets
> > > lost.
> > >
> > > Christopher
> > >
> > > smtp_new: incoming client on listener: 0x3c00ad40
> > > session_pickup: greeting client
> > > imsg: PROC_CONTROL <- PROC_SMTP: IMSG_SMTP_ENQUEUE (len=0)
> > > command: EHLO args: localhost
> > > imsg: PROC_MFA <- PROC_SMTP: IMSG_MFA_HELO (len=8128)
> > > imsg: PROC_SMTP <- PROC_MFA: IMSG_MFA_HELO (len=9428)
> > > command: MAIL FROM args: <madro...@alix.ftp.sh>
> > > session_rfc5321_mail_handler: sending notification to mfa
> > > imsg: PROC_MFA <- PROC_SMTP: IMSG_MFA_MAIL (len=8128)
> > > imsg: PROC_LKA <- PROC_MFA: IMSG_LKA_MAIL (len=9428)
> > > imsg: PROC_MFA <- PROC_LKA: IMSG_LKA_MAIL (len=9428)
> > > imsg: PROC_SMTP <- PROC_MFA: IMSG_MFA_MAIL (len=9428)
> > > imsg: PROC_QUEUE <- PROC_SMTP: IMSG_QUEUE_CREATE_MESSAGE (len=8128)
> > > imsg: PROC_SMTP <- PROC_QUEUE: IMSG_QUEUE_CREATE_MESSAGE (len=9428)
> > > command: RCPT TO args: <madro...@alix.ftp.sh>
> > > imsg: PROC_MFA <- PROC_SMTP: IMSG_MFA_RCPT (len=8128)
> > > imsg: PROC_LKA <- PROC_MFA: IMSG_LKA_RULEMATCH (len=9428)
> > > imsg: PROC_MFA <- PROC_LKA: IMSG_LKA_RULEMATCH (len=9428)
> > > imsg: PROC_LKA <- PROC_MFA: IMSG_LKA_RCPT (len=9428)
> > > aliases_exist: 'madroach' exists with 1 expansion nodes
> > > aliases_get: returned 1 aliases
> > > lka_resolve_node: node is filter: true
> > > imsg: PROC_QUEUE <- PROC_LKA: IMSG_QUEUE_SUBMIT_ENVELOPE (len=8128)
> > > imsg: PROC_QUEUE <- PROC_LKA: IMSG_QUEUE_COMMIT_ENVELOPES (len=8128)
> > > imsg: PROC_SMTP <- PROC_QUEUE: IMSG_QUEUE_COMMIT_ENVELOPES
> > > (len=9428) command: DATA args: (null)
> > > imsg: PROC_SMTP <- PROC_QUEUE: IMSG_QUEUE_MESSAGE_FILE (len=9428)
> > > imsg: PROC_QUEUE <- PROC_SMTP: IMSG_QUEUE_COMMIT_MESSAGE (len=8128)
> > > imsg: PROC_RUNNER <- PROC_QUEUE: IMSG_QUEUE_COMMIT_MESSAGE
> > > (len=8128) imsg: PROC_SMTP <- PROC_QUEUE: IMSG_QUEUE_COMMIT_MESSAGE
> > > (len=9428) 5c8a626e: from=<madro...@alix.ftp.sh>, size=349,
> > > nrcpts=1, proto=ESMTP, relay=1000@localhost [IPv6:::1]
> > > command: QUIT args: (null)
> > > session_destroy: killing client: 0x89a3c000
> > > runner: nothing to schedule, wake me up. zZzZzZ
> > > imsg: PROC_QUEUE <- PROC_RUNNER: IMSG_MDA_SESS_NEW (len=8128)
> > > imsg: PROC_MDA <- PROC_QUEUE: IMSG_MDA_SESS_NEW (len=8128)
> > > imsg: PROC_PARENT <- PROC_MDA: IMSG_PARENT_FORK_MDA (len=1058)
> > > forkmda: to true as
> > > imsg: PROC_MDA <- PROC_PARENT: IMSG_MDA_DONE (len=23)
> > > 5c8a626ea8724c2d: to=<madro...@alix.ftp.sh>, delay=0, stat=Error
> > > (getpwnam: no such user)
> >
>
> --
> Gilles Chehade
>
> https://www.poolp.org |
> http://pool.ps @poolpOrg
>
--
Gilles Chehade
https://www.poolp.org | http://pool.ps @poolpOrg
