Explicitly use local scope of function variables for safety

[govpn.git] / utils / addroute.sh

diff --git a/utils/addroute.sh b/utils/addroute.sh
index 743bbd597d0d8ffc681761d2f6faa818c8133c0b..781505c95d4c123994a00d8ae92da25ed3ae71d9 100755 (executable)
--- a/utils/addroute.sh
+++ b/utils/addroute.sh
@@ -1,4 +1,4 @@
-#!/bin/bash -x
+#!/bin/sh -x
 
 # A simple script handling default routing for GoVPN,
 # inspired by vpnc-script, but much simpler.
@@ -23,7 +23,7 @@ tear_down_dev() {
 
 
 do_connect() {
-  OLDGW=$(ip route show 0/0 | sed -e 's/^default//')
+  local OLDGW=$(ip route show 0/0 | sed -e 's/^default//')
   ip link set dev $TAPDEV up
   ip addr add $INTERNAL_IP4_ADDRESS dev $TAPDEV
   ip route add $VPNGATEWAY $OLDGW