diff options
author | Felix Fietkau <nbd@openwrt.org> | 2006-10-24 23:59:08 +0000 |
---|---|---|
committer | Felix Fietkau <nbd@openwrt.org> | 2006-10-24 23:59:08 +0000 |
commit | 315f8e954bf209cc2de0dfd423aaa0611afb869d (patch) | |
tree | 404cdd638c2d7667df57ed993e9aa3cccc9e55ca /package/base-files/default | |
parent | 2f78952da8a9be5ca46dc3b705aa517072c9cec6 (diff) | |
download | master-31e0f0ae-315f8e954bf209cc2de0dfd423aaa0611afb869d.tar.gz master-31e0f0ae-315f8e954bf209cc2de0dfd423aaa0611afb869d.tar.bz2 master-31e0f0ae-315f8e954bf209cc2de0dfd423aaa0611afb869d.zip |
make network scripts more robust, prevent multiple instances of pppd and udhcpc for the same interface
SVN-Revision: 5290
Diffstat (limited to 'package/base-files/default')
-rwxr-xr-x | package/base-files/default/lib/network/config.sh | 3 | ||||
-rwxr-xr-x | package/base-files/default/sbin/ifdown | 35 | ||||
-rwxr-xr-x | package/base-files/default/sbin/ifup | 5 |
3 files changed, 22 insertions, 21 deletions
diff --git a/package/base-files/default/lib/network/config.sh b/package/base-files/default/lib/network/config.sh index a66c92bc03..25c7278e99 100755 --- a/package/base-files/default/lib/network/config.sh +++ b/package/base-files/default/lib/network/config.sh @@ -120,8 +120,9 @@ setup_interface() { env -i ACTION="ifup" INTERFACE="config" DEVICE="$iface" PROTO=static /sbin/hotplug "iface" & ;; dhcp) + # prevent udhcpc from starting more than once pid="$(cat "$pidfile" 2>/dev/null)" - [ -n "$pid" -a -d "/proc/$pid" ] && kill -9 "$pid" + [ -d "/proc/$pid" ] && grep udhcpc "/proc/${pid}/cmdline" >/dev/null 2>/dev/null && return 0 config_get ipaddr "$config" ipaddr config_get netmask "$config" netmask diff --git a/package/base-files/default/sbin/ifdown b/package/base-files/default/sbin/ifdown index 073e80d2da..37d0918e85 100755 --- a/package/base-files/default/sbin/ifdown +++ b/package/base-files/default/sbin/ifdown @@ -12,23 +12,26 @@ debug "### ifdown $cfg ###" config_get proto "$cfg" proto [ -z "$proto" ] && { echo "interface not found."; exit; } -# kill active ppp daemon -pid="$(cat /var/run/ppp-${cfg}.pid 2>/dev/null)" -[ -n "$pid" -a -d "/proc/$pid" ] && { - kill $pid - sleep 1 - [ -d "/proc/$pid" ] && kill -9 $pid -} - -# kill any other process associated with the interface +# kill active ppp daemon and other processes config_get ifname "$cfg" ifname -pid="$(cat /var/run/${ifname}.pid 2>/dev/null)" -[ -n "$pid" -a -d "/proc/$pid" ] && kill -9 $pid +pids="$(cat /var/run/${ifname}.pid /var/run/ppp-${cfg}.pid 2>/dev/null)" +for pid in $pids; do + [ -d "/proc/$pid" ] && { + kill $pid + sleep 1 + [ -d "/proc/$pid" ] && kill -9 $pid + } +done +rm -f /var/run/${ifname}.pid /var/run/ppp-${cfg}.pid config_get ifname "$cfg" ifname -ifconfig "$ifname" >/dev/null 2>/dev/null && { - ifconfig "$ifname" 0.0.0.0 down +config_get device "$cfg" ifname + +[ ."$device" != ."$ifname" ] || device= +for dev in $ifname $device; do + ifconfig "$ifname" 0.0.0.0 down >/dev/null 2>/dev/null +done + +config_get iftype "$cfg" type +[ "$iftype" = "bridge" ] && brctl delbr "$ifname" >/dev/null 2>/dev/null - config_get iftype "$cfg" type - [ "$iftype" = "bridge" ] && brctl delbr "$ifname" -} diff --git a/package/base-files/default/sbin/ifup b/package/base-files/default/sbin/ifup index c7055d4807..81ec2c0890 100755 --- a/package/base-files/default/sbin/ifup +++ b/package/base-files/default/sbin/ifup @@ -4,10 +4,7 @@ . /sbin/ifdown "$@" config_get iftype "$1" type -case "$iftype" in - bridge) config_get ifname "$1" ifnames;; - *) config_get ifname "$1" ifname;; -esac +config_get ifname "$1" device for dev in $ifname; do setup_interface "$dev" "$1" |