summaryrefslogtreecommitdiff
path: root/shell
diff options
context:
space:
mode:
authorDenys Vlasenko2020-02-17 11:22:59 +0100
committerDenys Vlasenko2020-02-17 11:22:59 +0100
commitafc91faeddd6b8234dccea2f7913f57a5bb3d1ec (patch)
treeae4b2d8f7802cc94987841832d010ba3129773a1 /shell
parent9ee5892798be81f7a6f3e070ecd52cbf0d55740e (diff)
downloadbusybox-afc91faeddd6b8234dccea2f7913f57a5bb3d1ec.zip
busybox-afc91faeddd6b8234dccea2f7913f57a5bb3d1ec.tar.gz
ash: mkinit: Split reset into exitreset and reset
Upstream commit: Date: Sat, 19 May 2018 02:39:40 +0800 mkinit: Split reset into exitreset and reset Previously reset was called after exitshell. This was changed so that it was called before exitshell because certain state needed to be reset in order for the EXIT trap to work. However, this caused issues because certain other states (such as local variables) should not be reset. This patch fixes this by creating a new function exitreset that is called prior to exitshell and moving reset back to its original location. Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'shell')
-rw-r--r--shell/ash.c26
1 files changed, 19 insertions, 7 deletions
diff --git a/shell/ash.c b/shell/ash.c
index dfe6d1c..fbe8dd9 100644
--- a/shell/ash.c
+++ b/shell/ash.c
@@ -14284,11 +14284,11 @@ read_profile(const char *name)
/*
* This routine is called when an error or an interrupt occurs in an
- * interactive shell and control is returned to the main command loop.
- * (In dash, this function is auto-generated by build machinery).
+ * interactive shell and control is returned to the main command loop
+ * but prior to exitshell.
*/
static void
-reset(void)
+exitreset(void)
{
/* from eval.c: */
evalskip = 0;
@@ -14301,14 +14301,23 @@ reset(void)
/* from expand.c: */
ifsfree();
+ /* from redir.c: */
+ unwindredir(NULL);
+}
+
+/*
+ * This routine is called when an error or an interrupt occurs in an
+ * interactive shell and control is returned to the main command loop.
+ * (In dash, this function is auto-generated by build machinery).
+ */
+static void
+reset(void)
+{
/* from input.c: */
g_parsefile->left_in_buffer = 0;
g_parsefile->left_in_line = 0; /* clear input buffer */
popallfiles();
- /* from redir.c: */
- unwindredir(NULL);
-
/* from var.c: */
unwindlocalvars(NULL);
}
@@ -14356,13 +14365,16 @@ int ash_main(int argc UNUSED_PARAM, char **argv)
smallint e;
smallint s;
- reset();
+ exitreset();
e = exception_type;
s = state;
if (e == EXEXIT || s == 0 || iflag == 0 || shlvl) {
exitshell();
}
+
+ reset();
+
if (e == EXINT) {
newline_and_flush(stderr);
}