changeset 206:0241aaccfcdb

default listen_addresses: use 127.0.0.1 instead of `localhost' because `localhost' could be bound to some other IP address. This is unlikely but could be. Using 127.0.0.1 is more safe. See mailing list for details
author meillo@marmaro.de
date Sat, 17 Jul 2010 00:34:40 +0200 (2010-07-16)
parents 4fd237550525
children dcb315792513
files INSTALL man/masqmail.conf.5 src/conf.c
diffstat 3 files changed, 8 insertions(+), 4 deletions(-) [+]
line wrap: on
line diff
--- a/INSTALL	Fri Jul 16 16:38:50 2010 +0200
+++ b/INSTALL	Sat Jul 17 00:34:40 2010 +0200
@@ -164,8 +164,9 @@
 
 Such a setup (i.e. the default one) does:
 - deliver mail locally
-- accept mail from local (via stdin)
-- accept mail on localhost:25 (via SMTP) (if started as daemon)
+- accept mail on stdin (plain text)
+- accept mail on stdin (SMTP) (if started with -bs)
+- accept mail on the local port 25 (SMTP) (if started with -bd)
 
 It does not
 - transfer mail to other machines
--- a/man/masqmail.conf.5	Fri Jul 16 16:38:50 2010 +0200
+++ b/man/masqmail.conf.5	Sat Jul 17 00:34:40 2010 +0200
@@ -176,7 +176,7 @@
 If your host has different names which resolve to the same IP,
 use only one of them, otherwise you will get an error message.
 
-Default: \fIlocalhost:25\fR (i.e. only local processes can connect)
+Default: \fI127.0.0.1:25\fR (i.e. only local processes can connect)
 
 .TP
 \fBdo_save_envelope_to = \fIboolean\fR
--- a/src/conf.c	Fri Jul 16 16:38:50 2010 +0200
+++ b/src/conf.c	Sat Jul 17 00:34:40 2010 +0200
@@ -433,7 +433,10 @@
 	conf.max_msg_size = 0; /* no limit on msg size */
 	conf.spool_dir = SPOOL_DIR;
 	conf.mail_dir = "/var/mail";
-	conf.listen_addresses = g_list_append(NULL, parse_interface("localhost", 25));
+	/* we use 127.0.0.1 because `localhost' could be bound to some
+	   other IP address. This is unlikely but could be. Using
+	   127.0.0.1 is more safe. See mailing list for details */
+	conf.listen_addresses = g_list_append(NULL, parse_interface("127.0.0.1", 25));
 
 	if ((in = fopen(filename, "r")) == NULL) {
 		logwrite(LOG_ALERT, "could not open config file %s: %s\n", filename, strerror(errno));