fix ppp unit number issues (#2178)

[openwrt.git] / package / ppp / files / ppp.sh

diff --git a/package/ppp/files/ppp.sh b/package/ppp/files/ppp.sh
index e6008f4..5ca1cf9 100644 (file)
--- a/package/ppp/files/ppp.sh
+++ b/package/ppp/files/ppp.sh
@@ -1,30 +1,68 @@
 scan_ppp() {
        config_get ifname "$1" ifname
        pppdev="${pppdev:-0}"
 scan_ppp() {
        config_get ifname "$1" ifname
        pppdev="${pppdev:-0}"

-       config_set "$1" device "$ifname"
-       config_set "$1" ifname "ppp$pppdev"
-       config_set "$1" unit "$pppdev"
+       config_get unit "$1" unit
+       [ -z "$unit" ] && {
+               config_set "$1" ifname "ppp$pppdev"
+               config_set "$1" unit "$pppdev"
+               pppdev="$(($pppdev + 1))"
+       }

 }
 
 start_pppd() {
        local cfg="$1"; shift
 }
 
 start_pppd() {
        local cfg="$1"; shift

+
+       # make sure only one pppd process is started
+       lock "/var/lock/ppp-${cfg}"
+       local pid="$(head -n1 /var/run/ppp-${cfg}.pid 2>/dev/null)"
+       [ -d "/proc/$pid" ] && grep pppd "/proc/$pid/cmdline" 2>/dev/null >/dev/null && {
+               lock -u "/var/lock/ppp-${cfg}"
+               return 0
+       }
+

        config_get device "$cfg" device
        config_get unit "$cfg" unit
        config_get username "$cfg" username
        config_get password "$cfg" password
        config_get keepalive "$cfg" keepalive
        config_get device "$cfg" device
        config_get unit "$cfg" unit
        config_get username "$cfg" username
        config_get password "$cfg" password
        config_get keepalive "$cfg" keepalive

-       interval="${keepalive%%*,}"
+
+       config_get connect "$cfg" connect
+       config_get disconnect "$cfg" disconnect
+       config_get pppd_options "$cfg" pppd_options
+       config_get_bool defaultroute "$cfg" defaultroute 1
+       [ "$defaultroute" -eq 1 ] && defaultroute="defaultroute replacedefaultroute" || defaultroute=""
+
+       interval="${keepalive##*[, ]}"

        [ "$interval" != "$keepalive" ] || interval=5
        
        config_get demand "$cfg" demand
        [ "$interval" != "$keepalive" ] || interval=5
        
        config_get demand "$cfg" demand

-       [ -n "$demand" ] && echo "nameserver 1.1.1.1" > /tmp/resolv.conf
+       [ -n "$demand" ] && echo "nameserver 1.1.1.1" > /tmp/resolv.conf.auto

        /usr/sbin/pppd "$@" \
        /usr/sbin/pppd "$@" \

-               ${keepalive:+lcp-echo-interval $interval lcp-echo-failure ${keepalive##,*}} \
+               ${keepalive:+lcp-echo-interval $interval lcp-echo-failure ${keepalive%%[, ]*}} \

                ${demand:+precompiled-active-filter /etc/ppp/filter demand idle }${demand:-persist} \
                usepeerdns \
                ${demand:+precompiled-active-filter /etc/ppp/filter demand idle }${demand:-persist} \
                usepeerdns \

-               defaultroute \
-               replacedefaultroute \
+               $defaultroute \

                ${username:+user "$username" password "$password"} \
                ${username:+user "$username" password "$password"} \

+               unit "$unit" \

                linkname "$cfg" \
                linkname "$cfg" \

-               ipparam "$cfg"
+               ipparam "$cfg" \
+               ${connect:+connect "$connect"} \
+               ${disconnect:+disconnect "$disconnect"} \
+               ${pppd_options}
+
+       lock -u "/var/lock/ppp-${cfg}"

 }
 }

+
+setup_interface_ppp() {
+       local iface="$1"
+       local config="$2"
+
+       config_get device "$config" device
+
+       config_get mtu "$cfg" mtu
+       mtu=${mtu:-1492}
+       start_pppd "$config" \
+               mtu $mtu mru $mtu \
+               "$device"
+}
+