From: Tobias Brunner <tobias@strongswan.org>
Date: Mon, 17 Sep 2018 13:56:48 +0000 (+0200)
Subject: kernel-netlink: Check return value of both halfs when installing default route in... 
X-Git-Tag: 5.7.0rc2~19
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=0af96ad5e7e52f1f6f3500735c34d4a3b30cf58e;p=thirdparty%2Fstrongswan.git

kernel-netlink: Check return value of both halfs when installing default route in main table
---

diff --git a/src/libcharon/plugins/kernel_netlink/kernel_netlink_net.c b/src/libcharon/plugins/kernel_netlink/kernel_netlink_net.c
index c9a76ba01a..760a875ca5 100644
--- a/src/libcharon/plugins/kernel_netlink/kernel_netlink_net.c
+++ b/src/libcharon/plugins/kernel_netlink/kernel_netlink_net.c
@@ -2586,11 +2586,11 @@ static status_t manage_srcroute(private_kernel_netlink_net_t *this,
 		memset(half_net.ptr, 0, half_net.len);
 		half_prefixlen = 1;
 
-		status = manage_srcroute(this, nlmsg_type, flags, half_net, half_prefixlen,
-					gateway, src_ip, if_name);
+		status = manage_srcroute(this, nlmsg_type, flags, half_net,
+								 half_prefixlen, gateway, src_ip, if_name);
 		half_net.ptr[0] |= 0x80;
-		status = manage_srcroute(this, nlmsg_type, flags, half_net, half_prefixlen,
-					gateway, src_ip, if_name);
+		status |= manage_srcroute(this, nlmsg_type, flags, half_net,
+								  half_prefixlen, gateway, src_ip, if_name);
 		return status;
 	}