[PATCH] D124607: Add an error message to the default SIGPIPE handler

[Damian Malarczyk via Phabricator via llvm-commits]

dmcyk updated this revision to Diff 433346.
dmcyk added a comment.

- rebase
- Use llvm-cxxfilt to test SIGPIPE handler


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124607/new/

https://reviews.llvm.org/D124607

Files:
  llvm/lib/Support/Unix/Signals.inc
  llvm/test/tools/llvm-cxxfilt/unix03-sigpipe-exit.test


Index: llvm/test/tools/llvm-cxxfilt/unix03-sigpipe-exit.test
===================================================================
--- /dev/null
+++ llvm/test/tools/llvm-cxxfilt/unix03-sigpipe-exit.test
@@ -0,0 +1,26 @@
+# Test that when nm tries to write to a closed stdout it will finish with
+# a non-zero exit code and an error message on stderr.
+# This is required for UNIX03 conformance.
+
+# UNSUPPORTED: system-windows
+
+# RUN: not %python %s llvm-cxxfilt 2>&1 | FileCheck %s
+# CHECK: error: write on a pipe with no reader
+
+import subprocess
+import sys
+
+with subprocess.Popen([sys.argv[1]], stdout=subprocess.PIPE, stdin=subprocess.PIPE) as process:
+  process.stdout.close()
+
+  # llvm-cxxfilt with no extra arguments runs interactively and writes input 
+  # to output. Writing continuously to stdin should trigger SIGPIPE when the
+  # subprocess attempts to write out bytes to a closed stdout.
+  try:
+    while True: 
+      process.stdin.write("foo\n".encode("utf-8"))
+  except BrokenPipeError:
+    # Clear stdin, pipe is broken and closing it on cleanup will rise an exception 
+    process.stdin = None
+    pass
+sys.exit(process.returncode)
Index: llvm/lib/Support/Unix/Signals.inc
===================================================================
--- llvm/lib/Support/Unix/Signals.inc
+++ llvm/lib/Support/Unix/Signals.inc
@@ -432,6 +432,10 @@
 }
 
 void llvm::sys::DefaultOneShotPipeSignalHandler() {
+  // UNIX03 conformance requires a non-zero exit code and an error message
+  // to stderr when writing to a closed stdout fails.
+  errs() << "error: write on a pipe with no reader\n";
+
   // Send a special return code that drivers can check for, from sysexits.h.
   exit(EX_IOERR);
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D124607.433346.patch
Type: text/x-patch
Size: 1722 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220601/579ffd87/attachment.bin>