diff options
author | Elliott Hughes <enh@google.com> | 2016-04-21 18:18:05 -0700 |
---|---|---|
committer | Rob Landley <rob@landley.net> | 2016-04-23 13:33:03 -0500 |
commit | 072ea41682be093fd5fb25e7d6669a65a76144e6 (patch) | |
tree | 361a7bde262ba4bc8ff007f014749ac19eeb71c1 /lib | |
parent | 63eae537018a758b29f6c9bc39e3d36be1710179 (diff) | |
download | toybox-072ea41682be093fd5fb25e7d6669a65a76144e6.tar.gz |
Fix more to not append an extra newline.
More's sigatexit handler needs to distinguish between normal exit and exit
due to receipt of a signal.
Change tty_sigreset to look at the signal number too, so that pressing 'q'
to exit top doesn't cause its exit status to be 128.
Diffstat (limited to 'lib')
-rw-r--r-- | lib/interestingtimes.c | 2 | ||||
-rw-r--r-- | lib/xwrap.c | 4 |
2 files changed, 4 insertions, 2 deletions
diff --git a/lib/interestingtimes.c b/lib/interestingtimes.c index 8337ce8b..62670cbb 100644 --- a/lib/interestingtimes.c +++ b/lib/interestingtimes.c @@ -239,5 +239,5 @@ void tty_reset(void) void tty_sigreset(int i) { tty_reset(); - _exit(128+i); + _exit(i ? 128+i : 0); } diff --git a/lib/xwrap.c b/lib/xwrap.c index 0b1ab8e9..36a601c5 100644 --- a/lib/xwrap.c +++ b/lib/xwrap.c @@ -47,7 +47,9 @@ void xexit(void) // Call toys.xexit functions in reverse order added. while (toys.xexit) { // This is typecasting xexit->arg to a function pointer,then calling it. - ((void (*)(void))(toys.xexit->arg))(); + // Using the invalid signal number 0 lets the signal handlers distinguish + // an actual signal from a regular exit. + ((void (*)(int))(toys.xexit->arg))(0); free(llist_pop(&toys.xexit)); } |