d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

sh: Check readonly status for assignments on regular builtins.

Browse Source 
An error message is written, the builtin is not executed, nonzero exit
status is returned but the shell does not abort.

This was already checked for special builtins and external commands, with
the same consequences except that the shell aborts for special builtins.

Obtained from:	NetBSD
This commit is contained in:
Jilles Tjoelker 2011-01-01 13:26:18 +00:00
parent 5610751bf1
commit 850460c0f1
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=216870
4 changed files with 17 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
bin/sh/eval.c
View File

@ -997,8 +997,7 @@ evalcommand(union node *cmd, int flags, struct backcmd *backcmd)
*/
if (argc == 0 && !(flags & EV_BACKCMD))
cmdentry.special = 1;
if (cmdentry.special)
listsetvar(cmdenviron);
listsetvar(cmdenviron, cmdentry.special ? 0 : VNOSET);
if (argc > 0)
bltinsetlocale();
commandname = argv[0];

8
bin/sh/var.c
View File

@ -333,6 +333,8 @@ setvareq(char *s, int flags)
len = strchr(s, '=') - s;
error("%.*s: is read only", len, s);
}
if (flags & VNOSET)
return;
INTOFF;
if (vp->func && (flags & VNOFUNC) == 0)
@ -365,6 +367,8 @@ setvareq(char *s, int flags)
}
}
/* not found */
if (flags & VNOSET)
return;
vp = ckmalloc(sizeof (*vp));
vp->flags = flags;
vp->text = s;
@ -386,13 +390,13 @@ setvareq(char *s, int flags)
*/
void
listsetvar(struct strlist *list)
listsetvar(struct strlist *list, int flags)
{
struct strlist *lp;
INTOFF;
for (lp = list ; lp ; lp = lp->next) {
setvareq(savestr(lp->text), 0);
setvareq(savestr(lp->text), flags);
}
INTON;
}

3
bin/sh/var.h
View File

@ -45,6 +45,7 @@
#define VSTACK 0x10 /* text is allocated on the stack */
#define VUNSET 0x20 /* the variable is not set */
#define VNOFUNC 0x40 /* don't call the callback function */
#define VNOSET 0x80 /* do not set variable - just readonly test */
struct var {
@ -106,7 +107,7 @@ void initvar(void);
void setvar(const char *, const char *, int);
void setvareq(char *, int);
struct strlist;
void listsetvar(struct strlist *);
void listsetvar(struct strlist *, int);
char *lookupvar(const char *);
char *bltinlookup(const char *, int);
void bltinsetlocale(void);

8
tools/regression/bin/sh/errors/assignment-error2.0 Normal file
View File

@ -0,0 +1,8 @@
# $FreeBSD$
set -e
HOME=/
readonly HOME
cd /sbin
{ HOME=/bin cd; } 2>/dev/null || :
[ "$(pwd)" != /bin ]