summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Sommerseth <davids@redhat.com>2011-11-21 12:49:33 +0100
committerDavid Sommerseth <davids@redhat.com>2011-11-21 13:19:55 +0100
commit93626f2cf71b4803aa1248a928bfc1ac8c9da18b (patch)
tree877c379955c9cead592cc93120fc25fafe6eb349
parent354bc04286abe157f906309bb2ab463ac160c5a7 (diff)
downloadopenvpn-93626f2cf71b4803aa1248a928bfc1ac8c9da18b.tar.gz
openvpn-93626f2cf71b4803aa1248a928bfc1ac8c9da18b.tar.xz
openvpn-93626f2cf71b4803aa1248a928bfc1ac8c9da18b.zip
Fix FreeBSD/OpenBSD/NetBSD compiler warnings in get_default_gateway()
On these platforms (including DragonFly), get_default_gateway() would in some cases return false. As get_default_gateway() is defined as a void function, and none of the callers expect a return value -> just return without any value. Signed-off-by: David Sommerseth <davids@redhat.com> Acked-by: Gert Doering <gert@greenie.muc.de>
-rw-r--r--route.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/route.c b/route.c
index d632b45..be23a89 100644
--- a/route.c
+++ b/route.c
@@ -2581,7 +2581,7 @@ get_default_gateway (struct route_gateway_info *rgi)
warn("writing to routing socket");
gc_free (&gc);
close(s);
- return false;
+ return;
}
do {
@@ -2605,7 +2605,7 @@ get_default_gateway (struct route_gateway_info *rgi)
else
{
gc_free (&gc);
- return false;
+ return;
}
@@ -2947,7 +2947,7 @@ get_default_gateway (struct route_gateway_info *rgi)
warn("writing to routing socket");
gc_free (&gc);
close(s);
- return false;
+ return;
}
do {
@@ -2971,7 +2971,7 @@ get_default_gateway (struct route_gateway_info *rgi)
else
{
gc_free (&gc);
- return false;
+ return;
}