emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

sendmail: use host rather than NIS domain name for HELO

Browse Source 
According to RFC 5321 the argument to HELO "contains the fully-qualified
domain name of the SMTP client" or its IP address if no FQDN is available.
BusyBox sendmail uses the NIS domain name instead which, in many cases,
is likely to be the default "(none)". [vda: yes, I checked my machine
and its uts.domainname was indeed "(none)"]

Using the host name is more likely to satisfy the intent of the RFC while
allowing the otherwise unused safe_getdomainname function to be removed.

Signed-off-by: Ron Yorston <rmy@tigress.co.uk>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Ron Yorston
2012-04-28 17:04:19 +02:00
committed by Denys Vlasenko
parent 0c62f20454
commit 576b1d3c41
3 changed files with 4 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
mailutils/sendmail.c
View File

@ -118,7 +118,7 @@ int sendmail_main(int argc UNUSED_PARAM, char **argv)
char *opt_from;
char *s;
llist_t *list = NULL;
char *domain = sane_address(safe_getdomainname());
char *host = sane_address(safe_gethostname());
unsigned nheaders = 0;
int code;
@ -222,8 +222,9 @@ int sendmail_main(int argc UNUSED_PARAM, char **argv)
}
// we should start with modern EHLO
if (250 != smtp_checkp("EHLO %s", domain, -1))
smtp_checkp("HELO %s", domain, 250);
if (250 != smtp_checkp("EHLO %s", host, -1))
smtp_checkp("HELO %s", host, 250);
free(host);
// perform authentication
if (opts & OPT_a) {