From: tobhe <tobhe@openbsd.org>
Date: Thu, 4 Mar 2021 22:20:24 +0000 (+0000)
Subject: Derive config netmask from address pool if not explicitly configured.
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=c81386b3282b9d4bd74b5320abec48050dccb7d8;p=openbsd

Derive config netmask from address pool if not explicitly configured.

ok markus@
---

diff --git a/sbin/iked/Makefile b/sbin/iked/Makefile
index 387b74677ad..d9357857a2e 100644
--- a/sbin/iked/Makefile
+++ b/sbin/iked/Makefile
@@ -1,4 +1,4 @@
-# $OpenBSD: Makefile,v 1.18 2021/02/13 16:14:12 tobhe Exp $
+# $OpenBSD: Makefile,v 1.19 2021/03/04 22:20:24 tobhe Exp $
 
 PROG=		iked
 SRCS=		ca.c chap_ms.c config.c control.c crypto.c dh.c \
@@ -12,7 +12,7 @@ MAN=		iked.conf.5 iked.8
 
 LDADD=		-lutil -levent -lcrypto
 DPADD=		${LIBUTIL} ${LIBEVENT} ${LIBCRYPTO}
-CFLAGS+=	-Wall -I${.CURDIR}
+CFLAGS+=	-Wall -I${.CURDIR} -g
 CFLAGS+=	-Wstrict-prototypes -Wmissing-prototypes
 CFLAGS+=	-Wmissing-declarations
 CFLAGS+=	-Wshadow -Wpointer-arith -Wcast-qual
diff --git a/sbin/iked/ikev2.c b/sbin/iked/ikev2.c
index 7ba9b170e87..0c57242eb33 100644
--- a/sbin/iked/ikev2.c
+++ b/sbin/iked/ikev2.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: ikev2.c,v 1.310 2021/02/20 22:00:32 tobhe Exp $	*/
+/*	$OpenBSD: ikev2.c,v 1.311 2021/03/04 22:20:24 tobhe Exp $	*/
 
 /*
  * Copyright (c) 2019 Tobias Heider <tobias.heider@stusta.de>
@@ -2277,6 +2277,7 @@ ikev2_add_cp(struct iked *env, struct iked_sa *sa, int type, struct ibuf *buf)
 	struct sockaddr_in6	*in6;
 	uint8_t			 prefixlen;
 	int			 sent_addr4 = 0, sent_addr6 = 0;
+	int			 have_mask4 = 0, sent_mask4 = 0;
 
 	if ((cp = ibuf_advance(buf, sizeof(*cp))) == NULL)
 		return (-1);
@@ -2338,8 +2339,15 @@ ikev2_add_cp(struct iked *env, struct iked_sa *sa, int type, struct ibuf *buf)
 			if (ibuf_add(buf, &in4->sin_addr.s_addr, 4) == -1)
 				return (-1);
 			len += 4;
-			if (ikecfg->cfg_type == IKEV2_CFG_INTERNAL_IP4_ADDRESS)
+			if (ikecfg->cfg_type == IKEV2_CFG_INTERNAL_IP4_ADDRESS) {
 				sent_addr4 = 1;
+				if (sa->sa_addrpool &&
+				    sa->sa_addrpool->addr_af == AF_INET &&
+				    sa->sa_addrpool->addr_mask != 0)
+					have_mask4 = 1;
+			}
+			if (ikecfg->cfg_type == IKEV2_CFG_INTERNAL_IP4_NETMASK)
+				sent_mask4 = 1;
 			break;
 		case IKEV2_CFG_INTERNAL_IP4_SUBNET:
 			/* 4 bytes IPv4 address + 4 bytes IPv4 mask + */
@@ -2396,6 +2404,19 @@ ikev2_add_cp(struct iked *env, struct iked_sa *sa, int type, struct ibuf *buf)
 		}
 	}
 
+	/* derive netmask from pool */
+	if (type == IKEV2_CP_REPLY && have_mask4 && !sent_mask4) {
+		if ((cfg = ibuf_advance(buf, sizeof(*cfg))) == NULL)
+			return (-1);
+		cfg->cfg_type = htobe16(IKEV2_CFG_INTERNAL_IP4_NETMASK);
+		len += sizeof(*cfg);
+		mask4 = prefixlen2mask(sa->sa_addrpool->addr_mask);
+		cfg->cfg_length = htobe16(4);
+		if (ibuf_add(buf, &mask4, 4) == -1)
+			return (-1);
+		len += 4;
+	}
+
 	return (len);
 }
 
@@ -6924,6 +6945,7 @@ ikev2_cp_setaddr_pool(struct iked *env, struct iked_sa *sa,
 		}
 	}
 
+	addr.addr_mask = ikecfg->cfg.address.addr_mask;
 	switch (addr.addr_af) {
 	case AF_INET:
 		if (!key.sa_addrpool)