diff options
| author | Selva Nair | 2023-03-22 11:15:50 -0400 |
|---|---|---|
| committer | Gert Doering | 2023-03-22 16:59:58 +0100 |
| commit | b8ee8c43fd81988a58343da466999d2e30ae6221 (patch) | |
| tree | f21cc7d0f980b979877d500cc62122d3781febf8 /src/openvpnserv/interactive.c | |
| parent | 6c64b46b15476351ca19f9a8f3cb8185aa2c7e07 (diff) | |
| download | openvpn-b8ee8c43fd81988a58343da466999d2e30ae6221.zip openvpn-b8ee8c43fd81988a58343da466999d2e30ae6221.tar.gz | |
Make error in setting metric for IPv6 interface non-fatal
- Unfortunately there are still users out there who disable IPv6
on tun/tap/dco interfaces or even system-wide.
Github: fixes OpenVPN/openvpn#294
Signed-off-by: Selva Nair <selva.nair@gmail.com>
Acked-by: Gert Doering <gert@greenie.muc.de>
Message-Id: <20230322151550.1596669-1-selva.nair@gmail.com>
URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg26477.html
Signed-off-by: Gert Doering <gert@greenie.muc.de>
Diffstat (limited to 'src/openvpnserv/interactive.c')
| -rw-r--r-- | src/openvpnserv/interactive.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/src/openvpnserv/interactive.c b/src/openvpnserv/interactive.c index a3d4375..ec19627 100644 --- a/src/openvpnserv/interactive.c +++ b/src/openvpnserv/interactive.c @@ -847,8 +847,13 @@ AddBlockDNS(const block_dns_message_t *msg, undo_lists_t *lists) BLOCK_DNS_IFACE_METRIC); if (!err) { - err = set_interface_metric(msg->iface.index, AF_INET6, - BLOCK_DNS_IFACE_METRIC); + /* for IPv6, we intentionally ignore errors, because + * otherwise block-dns activation will fail if a user or + * admin has disabled IPv6 on the tun/tap/dco interface + * (if OpenVPN wants IPv6 ifconfig, we'll fail there) + */ + set_interface_metric(msg->iface.index, AF_INET6, + BLOCK_DNS_IFACE_METRIC); } if (err) { |