don't rely on temporary variables in config() as they may get overwritten by config_cb()

[openwrt.git] / package / base-files / default / etc / functions.sh

diff --git a/package/base-files/default/etc/functions.sh b/package/base-files/default/etc/functions.sh
index fbb5a16..3010196 100755 (executable)
--- a/package/base-files/default/etc/functions.sh
+++ b/package/base-files/default/etc/functions.sh
@@ -5,6 +5,8 @@ alias debug=${DEBUG:-:}
 N="
 "
 
+_C=0
+
 # valid interface?
 if_valid () (
   ifconfig "$1" >&- 2>&- ||
@@ -23,17 +25,21 @@ append() {
        eval "export ${var}=\"\${${var}:+\${${var}}${value:+$sep}}$value\""
 }
 
-config_cb() {
-       return 0
-}
-option_cb() {
-       return 0
+reset_cb() {
+       config_cb() {
+               return 0
+       }
+       option_cb() {
+               return 0
+       }
 }
+reset_cb
 
 config () {
-       config_cb "$@"
-       _C=$((${_C:-0} + 1))
-       export CONFIG_SECTION="${2:-cfg${_C}}"
+       _C=$(($_C + 1))
+       name="${name:-cfg${_C}}"
+       config_cb "$1" "$2"
+       export CONFIG_SECTION="$2"
        export CONFIG_${CONFIG_SECTION}_TYPE="$1"
 }
 
@@ -53,12 +59,16 @@ config_clear() {
 }
 
 config_load() {
-       local CD=""
-       if [ \! -e "$1" -a -e "/etc/config/$1" ]; then
-               cd /etc/config && local CD=1
-       fi
-       [ -e "$1" ] && . $1
-       ${CD:+cd - >/dev/null}
+       local DIR="./"
+       _C=0
+       [ \! -e "$1" -a -e "/etc/config/$1" ] && {
+               DIR="/etc/config/"
+       }
+       [ -e "$DIR$1" ] && {
+               CONFIG_FILENAME="$DIR$1"
+               . ${CONFIG_FILENAME}
+       } || return 1
+       ${CD:+cd -} >/dev/null
        ${CONFIG_SECTION:+config_cb}
 }