diff options
author | John Crispin <blogic@openwrt.org> | 2013-11-25 14:09:16 +0000 |
---|---|---|
committer | John Crispin <blogic@openwrt.org> | 2013-11-25 14:09:16 +0000 |
commit | 00f0744743dc85ad4e273500a15bb707f0739a6f (patch) | |
tree | e6a8dee49cc5d13934387cd1170654967459b0e4 | |
parent | 9cb1e0893fa3df4ec4f0a19eb01a724b11bad1d3 (diff) | |
download | master-187ad058-00f0744743dc85ad4e273500a15bb707f0739a6f.tar.gz master-187ad058-00f0744743dc85ad4e273500a15bb707f0739a6f.tar.bz2 master-187ad058-00f0744743dc85ad4e273500a15bb707f0739a6f.zip |
pppd: Place plugin-specific options after `plugin name`.
Before this patch, if we specify a plugin specific option through
`pppd_options` in /etc/config/network, e.g. `rp_pppoe_verbose 1`, pppd
would quit with the following error in log.
Wed Oct 9 09:42:58 2013 daemon.notice netifd: GORG (1689): /usr/sbin/pppd: unrecognized option 'rp_pppoe_verbose'
Wed Oct 9 09:42:58 2013 daemon.err pppd[1689]: unrecognized option 'rp_pppoe_verbose'
Wed Oct 9 09:42:58 2013 daemon.notice netifd: GORG (1689): pppd version 2.4.5
Wed Oct 9 09:42:58 2013 daemon.notice netifd: GORG (1689): Usage: /usr/sbin/pppd [ options ], where options are:
This is due to the requirement that function add_option() should be
called by the plugin_init() function first before pppd can parse those
options.
Signed-off-by: Yousong Zhou <yszhou4tech@gmail.com>
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@38911 3c298f89-4303-0410-b956-a3cf2f4a3e73
-rwxr-xr-x | package/network/services/ppp/files/ppp.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/package/network/services/ppp/files/ppp.sh b/package/network/services/ppp/files/ppp.sh index 02b40558ca..284d44cfed 100755 --- a/package/network/services/ppp/files/ppp.sh +++ b/package/network/services/ppp/files/ppp.sh @@ -55,7 +55,7 @@ ppp_generic_setup() { ip-down-script /lib/netifd/ppp-down \ ipv6-down-script /lib/netifd/ppp-down \ ${mtu:+mtu $mtu mru $mtu} \ - $pppd_options "$@" + "$@" $pppd_options } ppp_generic_teardown() { |