summaryrefslogtreecommitdiff
path: root/usr.sbin
diff options
context:
space:
mode:
authorEric Faurot <eric@cvs.openbsd.org>2018-09-17 12:16:28 +0000
committerEric Faurot <eric@cvs.openbsd.org>2018-09-17 12:16:28 +0000
commit5b4fdcb2dd64d8ac15bce1e211ea3b1c165e7de1 (patch)
tree51d47b6c1bb542e1b91bd90b73f7b3f8908170eb /usr.sbin
parent4ef84810aa19215d32dcfb4554c544ad7414d8c5 (diff)
simplify code path for backup relay and remove useless flag
ok gilles@
Diffstat (limited to 'usr.sbin')
-rw-r--r--usr.sbin/smtpd/mta.c18
-rw-r--r--usr.sbin/smtpd/mta_session.c5
-rw-r--r--usr.sbin/smtpd/smtpd.h3
3 files changed, 11 insertions, 15 deletions
diff --git a/usr.sbin/smtpd/mta.c b/usr.sbin/smtpd/mta.c
index c09fe4d71b4..638ccb958d5 100644
--- a/usr.sbin/smtpd/mta.c
+++ b/usr.sbin/smtpd/mta.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: mta.c,v 1.223 2018/09/08 10:05:07 eric Exp $ */
+/* $OpenBSD: mta.c,v 1.224 2018/09/17 12:16:27 eric Exp $ */
/*
* Copyright (c) 2008 Pierre-Yves Ritschard <pyr@openbsd.org>
@@ -1720,17 +1720,17 @@ mta_relay(struct envelope *e, struct relayhost *relayh)
key.helotable = dispatcher->u.remote.helo_source;
key.heloname = dispatcher->u.remote.helo;
- if (dispatcher->u.remote.backup) {
- key.backupname = dispatcher->u.remote.backupmx;
- if (key.backupname == NULL)
- key.backupname = e->smtpname;
- key.domain = mta_domain(e->dest.domain, 0);
- key.flags |= RELAY_BACKUP;
- } else if (relayh->hostname[0]) {
+ if (relayh->hostname[0]) {
key.domain = mta_domain(relayh->hostname, 1);
key.flags |= RELAY_MX;
- } else {
+ }
+ else {
key.domain = mta_domain(e->dest.domain, 0);
+ if (dispatcher->u.remote.backup) {
+ key.backupname = dispatcher->u.remote.backupmx;
+ if (key.backupname == NULL)
+ key.backupname = e->smtpname;
+ }
}
key.tls = relayh->tls;
diff --git a/usr.sbin/smtpd/mta_session.c b/usr.sbin/smtpd/mta_session.c
index 8b1db21f1a9..6cde788b55d 100644
--- a/usr.sbin/smtpd/mta_session.c
+++ b/usr.sbin/smtpd/mta_session.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: mta_session.c,v 1.110 2018/09/08 10:05:07 eric Exp $ */
+/* $OpenBSD: mta_session.c,v 1.111 2018/09/17 12:16:27 eric Exp $ */
/*
* Copyright (c) 2008 Pierre-Yves Ritschard <pyr@openbsd.org>
@@ -218,9 +218,6 @@ mta_session(struct mta_relay *relay, struct mta_route *route)
fatalx("bad value for relay->tls: %d", relay->tls);
}
- if (relay->flags & RELAY_BACKUP)
- s->flags &= ~MTA_FORCE_PLAIN;
-
log_debug("debug: mta: %p: spawned for relay %s", s,
mta_relay_to_text(relay));
stat_increment("mta.session", 1);
diff --git a/usr.sbin/smtpd/smtpd.h b/usr.sbin/smtpd/smtpd.h
index 108632f8dac..4e03177a45b 100644
--- a/usr.sbin/smtpd/smtpd.h
+++ b/usr.sbin/smtpd/smtpd.h
@@ -1,4 +1,4 @@
-/* $OpenBSD: smtpd.h,v 1.559 2018/09/08 10:05:07 eric Exp $ */
+/* $OpenBSD: smtpd.h,v 1.560 2018/09/17 12:16:27 eric Exp $ */
/*
* Copyright (c) 2008 Gilles Chehade <gilles@poolp.org>
@@ -90,7 +90,6 @@
#define RELAY_TLS_NO 3
#define RELAY_AUTH 0x08
-#define RELAY_BACKUP 0x10
#define RELAY_MX 0x20
#define RELAY_LMTP 0x80
#define RELAY_TLS_VERIFY 0x200