From: nbd <nbd@3c298f89-4303-0410-b956-a3cf2f4a3e73>
Date: Thu, 20 Mar 2014 14:14:30 +0000 (+0000)
Subject: busybox: fix existing "export -n" in ash
X-Git-Url: https://git.enpas.org/?a=commitdiff_plain;h=9f73ef41f8c5df3e2d3235286b4dd00ca5918f0e;p=openwrt.git

busybox: fix existing "export -n" in ash

Minor fix to existing patch which did not correctly implement -n support for the export command in ash. The VEXPORT flag of a variable was set in the first call to setvareq() and then remained set in the subsequent call, because the variable already existed in the environment.

Although /lib/functions.sh by default does not export variables (NO_EXPORT=1), this issue was leading to dirty environments.

Without this fix:
(ash) export A=1; export -n B=2; /bin/ash -c "echo \$A, \$B"
1, 2

With this fix:
(ash) export A=1; export -n B=2; /bin/ash -c "echo \$A, \$B"
1,

Signed-off-by: Christophe Lermytte <openwrt@lermytte.be>

git-svn-id: svn://svn.openwrt.org/openwrt/trunk@39968 3c298f89-4303-0410-b956-a3cf2f4a3e73
---

diff --git a/package/utils/busybox/patches/250-ash_export-n.patch b/package/utils/busybox/patches/250-ash_export-n.patch
index 048c236b85..ddbe74ee29 100644
--- a/package/utils/busybox/patches/250-ash_export-n.patch
+++ b/package/utils/busybox/patches/250-ash_export-n.patch
@@ -19,7 +19,7 @@
  		aptr = argptr;
  		name = *aptr;
  		if (name) {
-@@ -12624,10 +12633,12 @@ exportcmd(int argc UNUSED_PARAM, char **
+@@ -12624,10 +12633,11 @@ exportcmd(int argc UNUSED_PARAM, char **
  					vp = *findvar(hashvar(name), name);
  					if (vp) {
  						vp->flags |= flag;
@@ -27,7 +27,7 @@
  						continue;
  					}
  				}
- 				setvar(name, p, flag);
+-				setvar(name, p, flag);
 +				setvar(name, p, flag & mask);
  			} while ((name = *++aptr) != NULL);
  			return 0;