aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFelix Fietkau <nbd@openwrt.org>2015-04-17 00:01:29 +0000
committerFelix Fietkau <nbd@openwrt.org>2015-04-17 00:01:29 +0000
commit0d031aecdf66d10a2051726299d9930fbe098124 (patch)
treec01d49b6dae44a31ad6eba5a22e3f2426fbd705d
parente85294a957970b2cc50349bb3ec3f4c0113e22fd (diff)
downloadupstream-0d031aecdf66d10a2051726299d9930fbe098124.tar.gz
upstream-0d031aecdf66d10a2051726299d9930fbe098124.tar.bz2
upstream-0d031aecdf66d10a2051726299d9930fbe098124.zip
busybox: fix an ash regression in handling local variables
Signed-off-by: Felix Fietkau <nbd@openwrt.org> SVN-Revision: 45471
-rw-r--r--package/utils/busybox/patches/290-ash-fix-a-regression-in-handling-local-variables.patch75
1 files changed, 75 insertions, 0 deletions
diff --git a/package/utils/busybox/patches/290-ash-fix-a-regression-in-handling-local-variables.patch b/package/utils/busybox/patches/290-ash-fix-a-regression-in-handling-local-variables.patch
new file mode 100644
index 0000000000..0ac1925f3d
--- /dev/null
+++ b/package/utils/busybox/patches/290-ash-fix-a-regression-in-handling-local-variables.patch
@@ -0,0 +1,75 @@
+From: Felix Fietkau <nbd@openwrt.org>
+Date: Fri, 17 Apr 2015 01:54:51 +0200
+Subject: [PATCH] ash: fix a regression in handling local variables
+
+commit 109ee5d33694a03cda3424b4846584250832ba8e
+"ash: make "locak VAR" unset VAR (bash does that)"
+
+This commit introduced a regression where calling local on an already
+local variable unsets it. This does not match bash behavior.
+
+Update test case to check for this behavior
+
+Signed-off-by: Felix Fietkau <nbd@openwrt.org>
+---
+
+--- a/shell/ash.c
++++ b/shell/ash.c
+@@ -8961,6 +8961,21 @@ parse_command_args(char **argv, const ch
+ }
+ #endif
+
++static bool
++findlocal(struct var *vp)
++{
++ struct localvar *lvp = localvars;
++
++ while (lvp) {
++ if (lvp->vp == vp)
++ return true;
++
++ lvp = lvp->next;
++ }
++
++ return false;
++}
++
+ /*
+ * Make a variable a local variable. When a variable is made local, it's
+ * value and flags are saved in a localvar structure. The saved values
+@@ -9000,7 +9015,7 @@ mklocal(char *name)
+ vp->flags |= VSTRFIXED|VTEXTFIXED;
+ if (eq)
+ setvareq(name, 0);
+- else
++ else if (!findlocal(vp))
+ /* "local VAR" unsets VAR: */
+ setvar(name, NULL, 0);
+ }
+--- a/shell/ash_test/ash-misc/local1.right
++++ b/shell/ash_test/ash-misc/local1.right
+@@ -1,4 +1,5 @@
+ A1:'A'
+ A2:''
+-A3:''
+-A4:'A'
++A3:'B'
++A4:''
++A5:'A'
+--- a/shell/ash_test/ash-misc/local1.tests
++++ b/shell/ash_test/ash-misc/local1.tests
+@@ -3,9 +3,12 @@ f() {
+ local a
+ # the above line unsets $a
+ echo "A2:'$a'"
+- unset a
++ a=B
++ local a
+ echo "A3:'$a'"
++ unset a
++ echo "A4:'$a'"
+ }
+ echo "A1:'$a'"
+ f
+-echo "A4:'$a'"
++echo "A5:'$a'"