102/376: Restore default SIGPIPE handler before invoking ‘man’

[Ludovic Courtès]

civodul pushed a commit to tag 1.8
in repository guix.

commit 50dc1f5b71f7e0e4ba628904b9a47d9f9d875b3e
Author: Eelco Dolstra <address@hidden>
Date:   Thu Jul 31 10:31:17 2014 +0200

    Restore default SIGPIPE handler before invoking ‘man’
    
    Fixes NixOS/nixpkgs#3410.
---
 src/libmain/shared.cc |    6 +++---
 src/libstore/build.cc |   12 ------------
 src/libutil/util.cc   |   10 ++++++++++
 src/libutil/util.hh   |    4 ++++
 4 files changed, 17 insertions(+), 15 deletions(-)

diff --git a/src/libmain/shared.cc b/src/libmain/shared.cc
index 1426344..47c38e3 100644
--- a/src/libmain/shared.cc
+++ b/src/libmain/shared.cc
@@ -244,9 +244,9 @@ static void initAndRun(int argc, char * * argv)
 
 void showManPage(const string & name)
 {
-    string cmd = "man " + name;
-    if (system(cmd.c_str()) != 0)
-        throw Error(format("command `%1%' failed") % cmd);
+    restoreSIGPIPE();
+    execlp("man", "man", name.c_str(), NULL);
+    throw SysError(format("command `man %1%' failed") % name.c_str());
 }
 
 
diff --git a/src/libstore/build.cc b/src/libstore/build.cc
index 4376a83..479bc4a 100644
--- a/src/libstore/build.cc
+++ b/src/libstore/build.cc
@@ -413,18 +413,6 @@ const char * * strings2CharPtrs(const Strings & ss)
 }
 
 
-/* Restore default handling of SIGPIPE, otherwise some programs will
-   randomly say "Broken pipe". */
-static void restoreSIGPIPE()
-{
-    struct sigaction act, oact;
-    act.sa_handler = SIG_DFL;
-    act.sa_flags = 0;
-    sigemptyset(&act.sa_mask);
-    if (sigaction(SIGPIPE, &act, &oact)) throw SysError("resetting SIGPIPE");
-}
-
-
 //////////////////////////////////////////////////////////////////////
 
 
diff --git a/src/libutil/util.cc b/src/libutil/util.cc
index aa9d99e..f762b79 100644
--- a/src/libutil/util.cc
+++ b/src/libutil/util.cc
@@ -927,6 +927,16 @@ void closeOnExec(int fd)
 }
 
 
+void restoreSIGPIPE()
+{
+    struct sigaction act, oact;
+    act.sa_handler = SIG_DFL;
+    act.sa_flags = 0;
+    sigemptyset(&act.sa_mask);
+    if (sigaction(SIGPIPE, &act, &oact)) throw SysError("resetting SIGPIPE");
+}
+
+
 //////////////////////////////////////////////////////////////////////
 
 
diff --git a/src/libutil/util.hh b/src/libutil/util.hh
index ad0d377..64250c5 100644
--- a/src/libutil/util.hh
+++ b/src/libutil/util.hh
@@ -273,6 +273,10 @@ void closeMostFDs(const set<int> & exceptions);
 /* Set the close-on-exec flag for the given file descriptor. */
 void closeOnExec(int fd);
 
+/* Restore default handling of SIGPIPE, otherwise some programs will
+   randomly say "Broken pipe". */
+void restoreSIGPIPE();
+
 
 /* User interruption. */