make network scripts more robust, prevent multiple instances of pppd and udhcpc for the same interface
SVN-Revision: 5290
This commit is contained in:
parent
2f78952da8
commit
315f8e954b
4 changed files with 27 additions and 21 deletions
|
@ -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
|
||||
|
|
|
@ -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"
|
||||
}
|
||||
|
|
|
@ -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"
|
||||
|
|
|
@ -7,6 +7,11 @@ scan_ppp() {
|
|||
|
||||
start_pppd() {
|
||||
local cfg="$1"; shift
|
||||
|
||||
# make sure only one pppd process is started
|
||||
local pid="$(cat /var/run/ppp-${cfg}.pid 2>/dev/null)"
|
||||
[ -d "/proc/$pid" ] && grep pppd "/proc/$pid/cmdline" 2>/dev/null >/dev/null && return 0
|
||||
|
||||
config_get device "$cfg" device
|
||||
config_get unit "$cfg" unit
|
||||
config_get username "$cfg" username
|
||||
|
|
Loading…
Reference in a new issue