OSDN Git Service

Fix the comment about the Android SIGPIPE behavior.
authorElliott Hughes <enh@google.com>
Fri, 17 Feb 2017 22:43:56 +0000 (14:43 -0800)
committerRob Landley <rob@landley.net>
Sat, 18 Feb 2017 04:29:03 +0000 (22:29 -0600)
main.c

diff --git a/main.c b/main.c
index a2267da..d05cbfe 100644 (file)
--- a/main.c
+++ b/main.c
@@ -213,8 +213,12 @@ int main(int argc, char *argv[])
   }
   *argv = getbasename(*argv);
 
-  // Bionic's dynamic linker adds a handler to report SIGPIPE as an error,
-  // then doesn't want that behavior for toybox. So disable it for bionic.
+  // Up to and including Android M, bionic's dynamic linker added a handler to
+  // cause a crash dump on SIGPIPE. That was removed in Android N, but adbd
+  // was still setting the SIGPIPE disposition to SIG_IGN, and its children
+  // were inheriting that. In Android O, adbd is fixed, but manually asking
+  // for the default disposition is harmless, and it'll be a long time before
+  // no one's using anything older than O!
   if (CFG_TOYBOX_ON_ANDROID) signal(SIGPIPE, SIG_DFL);
 
   // If nommu can't fork, special reentry path.