[Toybox] [PATCH] more: don't _exit(0) on success.

enh enh at google.com
Mon Jan 7 22:38:27 PST 2019


If we do, we won't flush, and we might not output everything.
---
 toys/pending/more.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/toys/pending/more.c b/toys/pending/more.c
index 7923ee4e..fa96d40e 100644
--- a/toys/pending/more.c
+++ b/toys/pending/more.c
@@ -28,13 +28,13 @@ static void signal_handler(int sig)
   // Reset the terminal whether we were signalled or exited normally.
   tcsetattr(TT.cin_fd, TCSANOW, &TT.inf);

-  if (sig == 0) _exit(0);
-
-  // We were actually signalled, so move to a new line and re-raise the signal.
-  xputc('\n');
-  signal(sig, SIG_DFL);
-  raise(sig);
-  _exit(sig | 128);
+  // If we were actually signalled, move to a new line and re-raise the signal.
+  if (sig != 0) {
+    xputc('\n');
+    signal(sig, SIG_DFL);
+    raise(sig);
+    _exit(sig | 128);
+  }
 }

 static void show_file_header(const char *name)
-- 
2.20.1.97.g81188d93c3-goog
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-more-don-t-_exit-0-on-success.patch
Type: text/x-patch
Size: 1127 bytes
Desc: not available
URL: <http://lists.landley.net/pipermail/toybox-landley.net/attachments/20190107/7fa5dc4d/attachment-0002.bin>


More information about the Toybox mailing list