Use local variables in getconf & printconf.
authorMatthijs Kooijman <matthijs@stdin.nl>
Thu, 18 Mar 2010 09:35:49 +0000 (10:35 +0100)
committerMatthijs Kooijman <matthijs@stdin.nl>
Fri, 19 Mar 2010 20:40:15 +0000 (21:40 +0100)
lib/tools.in

index 2715a00643f4a2477c93378705191f8a28c47619..9879e741039a5994bd544fd8e60896c9612d6cd5 100644 (file)
@@ -31,8 +31,8 @@ function setsection() {
 # file and section and echoes its value. If it is empty or not found, $2
 # is used.
 function printconf() {
-   CURRENT_PARAM=$1
-   ret=`@AWK@ -f $libdirectory/parseini S=$CURRENT_SECTION P=$CURRENT_PARAM $CURRENT_CONF_FILE`
+   local CURRENT_PARAM=$1
+   local ret=`@AWK@ -f $libdirectory/parseini S=$CURRENT_SECTION P=$CURRENT_PARAM $CURRENT_CONF_FILE`
    # if nothing is returned, set the default
    if [ "$ret" == "" -a "$2" != "" ]; then
       ret="$2"
@@ -48,7 +48,7 @@ function printconf() {
 # same name. If it is empty or not found, $2 is used.
 #
 function getconf() {
-   ret=`printconf "$1" "$2"`
+   local ret=`printconf "$1" "$2"`
 
    # We use escape the $ in $ret to delay expansion of $ret, so when $1
    # is foo, eval sees foo=$ret and properly does the assignment