summaryrefslogtreecommitdiffstats
path: root/bin
diff options
context:
space:
mode:
authorcracauer <cracauer@FreeBSD.org>2000-08-16 10:39:43 +0000
committercracauer <cracauer@FreeBSD.org>2000-08-16 10:39:43 +0000
commit0a28a3436ddef5b200270a23d8de395cef74d655 (patch)
tree25ff2d6471b9dfc28f2b94134b41c648330b1304 /bin
parentfd45895cebe5b6c25dc13992350f3bf2a65d752a (diff)
downloadFreeBSD-src-0a28a3436ddef5b200270a23d8de395cef74d655.zip
FreeBSD-src-0a28a3436ddef5b200270a23d8de395cef74d655.tar.gz
From submitter:
growstackblock() sometimes relocates a stack_block considered empty without properly relocating stack marks referencing that block. The first call to popstackmark() with the unrelocated stack mark as argument then causes sh to abort. Relocating the relevant stack marks seems to solve this problem. The patch changes the semantics of popstackmark() somewhat. It can only be called once after a call to setstackmark(), thus cmdloop() in main.c needs an extra call to setstackmark(). PR: bin/19983 Submitted by: Tor.Egge@fast.no Reviewed by: Gerald Pfeifer <pfeifer@dbai.tuwien.ac.at>
Diffstat (limited to 'bin')
-rw-r--r--bin/sh/main.c3
-rw-r--r--bin/sh/memalloc.c19
-rw-r--r--bin/sh/memalloc.h1
3 files changed, 22 insertions, 1 deletions
diff --git a/bin/sh/main.c b/bin/sh/main.c
index 32eb82a..7d00586 100644
--- a/bin/sh/main.c
+++ b/bin/sh/main.c
@@ -253,12 +253,13 @@ cmdloop(top)
evaltree(n, 0);
}
popstackmark(&smark);
+ setstackmark(&smark);
if (evalskip == SKIPFILE) {
evalskip = 0;
break;
}
}
- popstackmark(&smark); /* unnecessary */
+ popstackmark(&smark);
}
diff --git a/bin/sh/memalloc.c b/bin/sh/memalloc.c
index b2155ff..c106775 100644
--- a/bin/sh/memalloc.c
+++ b/bin/sh/memalloc.c
@@ -118,6 +118,7 @@ struct stack_block {
struct stack_block stackbase;
struct stack_block *stackp = &stackbase;
+struct stackmark *markp;
char *stacknxt = stackbase.space;
int stacknleft = MINSIZE;
int sstrnleft;
@@ -176,6 +177,8 @@ setstackmark(mark)
mark->stackp = stackp;
mark->stacknxt = stacknxt;
mark->stacknleft = stacknleft;
+ mark->marknext = markp;
+ markp = mark;
}
@@ -186,6 +189,7 @@ popstackmark(mark)
struct stack_block *sp;
INTOFF;
+ markp = mark->marknext;
while (stackp != mark->stackp) {
sp = stackp;
stackp = sp->prev;
@@ -215,6 +219,7 @@ growstackblock()
char *oldspace;
int oldlen;
struct stack_block *sp;
+ struct stack_block *oldstackp;
newlen = ALIGN(stacknleft * 2 + 100);
oldspace = stacknxt;
@@ -222,6 +227,7 @@ growstackblock()
if (stacknxt == stackp->space && stackp != &stackbase) {
INTOFF;
+ oldstackp = stackp;
sp = stackp;
stackp = sp->prev;
sp = ckrealloc((pointer)sp, sizeof(struct stack_block) -
@@ -230,6 +236,19 @@ growstackblock()
stackp = sp;
stacknxt = sp->space;
stacknleft = newlen;
+ {
+ /* Stack marks pointing to the start of the old block
+ * must be relocated to point to the new block
+ */
+ struct stackmark *xmark;
+ xmark = markp;
+ while (xmark != NULL && xmark->stackp == oldstackp) {
+ xmark->stackp = stackp;
+ xmark->stacknxt = stacknxt;
+ xmark->stacknleft = stacknleft;
+ xmark = xmark->marknext;
+ }
+ }
INTON;
} else {
p = stalloc(newlen);
diff --git a/bin/sh/memalloc.h b/bin/sh/memalloc.h
index ab53a83..e3ca8be 100644
--- a/bin/sh/memalloc.h
+++ b/bin/sh/memalloc.h
@@ -41,6 +41,7 @@ struct stackmark {
struct stack_block *stackp;
char *stacknxt;
int stacknleft;
+ struct stackmark *marknext;
};
OpenPOWER on IntegriCloud