aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorArne Schwabe2023-07-12 11:46:20 +0200
committerGert Doering2023-07-17 18:49:08 +0200
commit4ed7d7f72515c7df34edf617cb0bb8f2ac533e48 (patch)
treedaa637a54626499d762d638da0b1259bc0470d44
parentd3fe78af5dde169fcb13a6fde3b532e5d1a3a633 (diff)
downloadopenvpn-4ed7d7f72515c7df34edf617cb0bb8f2ac533e48.zip
openvpn-4ed7d7f72515c7df34edf617cb0bb8f2ac533e48.tar.gz
Ignore Ipv6 route delete request on Android and set ipv4 verbosity to 7
Android has no facility nor need one to delete routes as routes are automatically cleaned up when the tun interface is closed. Also adjust the IPv4 message to be only shown and verb 7 and rephrase the message. Change-Id: If8f920d378c31e9ea773ce1f56f3df50f1ec36cd Signed-off-by: Arne Schwabe <arne@rfc2549.org> Acked-by: Gert Doering <gert@greenie.muc.de> Message-Id: <20230712094620.569273-1-arne@rfc2549.org> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg26848.html Signed-off-by: Gert Doering <gert@greenie.muc.de> (cherry picked from commit ab01eaf49fa9341ff647206bd6e3017770cc0674)
-rw-r--r--src/openvpn/route.c10
1 files changed, 7 insertions, 3 deletions
diff --git a/src/openvpn/route.c b/src/openvpn/route.c
index cb7cae0..f2852b4 100644
--- a/src/openvpn/route.c
+++ b/src/openvpn/route.c
@@ -2304,8 +2304,9 @@ delete_route(struct route_ipv4 *r,
openvpn_execve_check(&argv, es, 0, "ERROR: OpenBSD/NetBSD route delete command failed");
#elif defined(TARGET_ANDROID)
- msg(M_NONFATAL, "Sorry, deleting routes on Android is not possible. The VpnService API allows routes to be set on connect only.");
-
+ msg(D_ROUTE_DEBUG, "Deleting routes on Android is not possible/not "
+ "needed. The VpnService API allows routes to be set "
+ "on connect only and will clean up automatically.");
#elif defined(TARGET_AIX)
{
@@ -2492,7 +2493,10 @@ delete_route_ipv6(const struct route_ipv6 *r6, const struct tuntap *tt,
network, r6->netbits, gateway);
argv_msg(D_ROUTE, &argv);
openvpn_execve_check(&argv, es, 0, "ERROR: AIX route add command failed");
-
+#elif defined(TARGET_ANDROID)
+ msg(D_ROUTE_DEBUG, "Deleting routes on Android is not possible/not "
+ "needed. The VpnService API allows routes to be set "
+ "on connect only and will clean up automatically.");
#else /* if defined(TARGET_LINUX) */
msg(M_FATAL, "Sorry, but I don't know how to do 'route ipv6' commands on this operating system. Try putting your routes in a --route-down script");
#endif /* if defined(TARGET_LINUX) */