aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenys Vlasenko <vda.linux@googlemail.com>2017-07-17 21:31:17 +0200
committerDenys Vlasenko <vda.linux@googlemail.com>2017-07-17 21:31:17 +0200
commit6b48e1f1212300464c17c8317f5faddf6b414a4c (patch)
tree82df1620bf3da44cfa1db4c06066bf7a477215e9
parentb95ee96e7528554fc3ee2c48fbd59d75c59148db (diff)
downloadbusybox-6b48e1f1212300464c17c8317f5faddf6b414a4c.tar.gz
hush: forgot to emit error on (failing) second "readonly VAR=VAL"
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r--shell/hush.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/shell/hush.c b/shell/hush.c
index 2125e757d..1961c9830 100644
--- a/shell/hush.c
+++ b/shell/hush.c
@@ -2074,6 +2074,7 @@ static int set_local_var(char *str,
char *eq_sign;
int name_len;
+ //bb_error_msg("set_local_var('%s',%d,%d,%d)", str, flg_export, local_lvl, flg_read_only);
eq_sign = strchr(str, '=');
if (!eq_sign) { /* not expected to ever happen? */
free(str);
@@ -2090,8 +2091,7 @@ static int set_local_var(char *str,
/* We found an existing var with this name */
if (cur->flg_read_only) {
- if (!flg_read_only)
- bb_error_msg("%s: readonly variable", str);
+ bb_error_msg("%s: readonly variable", str);
free(str);
return -1;
}
@@ -9459,7 +9459,6 @@ static int FAST_FUNC builtin_readonly(char **argv)
}
#endif
-
#if ENABLE_HUSH_UNSET
/* http://www.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#unset */
static int FAST_FUNC builtin_unset(char **argv)