On Fri, Feb 12, 2016 at 04:29:23PM -0500, Peter Bisroev wrote: > Hi Gilles, >
Hi, > While looking over smtp_enqueue(), I have noticed that setting of > hostname is a noop. It looks like a leftover code from a bugfix in here > (http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin/smtpd/smtp.c.diff?r2=1.141&r1=1.140&f=u) > > I am including a diff to smtp.c below that includes the removal of that code > (it also includes the changes to smtp.c in my first patch). > Yes, it's a leftover from a cleanup to remove the euid from headers, it's a noop indeed. I've adapted to -current so it can be applied without your other diff (which is still pending review) and committed just a minute ago. Thanks ! -- Gilles Chehade https://www.poolp.org @poolpOrg
