ok, using signal.h for portability reasons

[dwm.git] / dwm.c

diff --git a/dwm.c b/dwm.c
index dc63a0b..378fde5 100644 (file)
--- a/dwm.c
+++ b/dwm.c
@@ -23,6 +23,7 @@
 #include <errno.h>
 #include <locale.h>
 #include <stdarg.h>
+#include <signal.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
@@ -178,6 +179,7 @@ static void setlayout(const Arg *arg);
 static void setmfact(const Arg *arg);
 static void setup(void);
 static void showhide(Client *c);
+static void sigchld(int signal);
 static void spawn(const Arg *arg);
 static void tag(const Arg *arg);
 static int textnw(const char *text, unsigned int len);
@@ -341,7 +343,6 @@ cleanup(void) {
        Arg a = {.ui = ~0};
        Layout foo = { "", NULL };
 
-       close(STDIN_FILENO);
        view(&a);
        lt[sellt] = &foo;
        while(stack)
@@ -1340,22 +1341,24 @@ showhide(Client *c) {
        }
 }
 
+
+void
+sigchld(int signal) {
+       while(0 < waitpid(-1, NULL, WNOHANG));
+}
+
 void
 spawn(const Arg *arg) {
-       /* The double-fork construct avoids zombie processes and keeps the code
-        * clean from stupid signal handlers. */
+       signal(SIGCHLD, sigchld);
        if(fork() == 0) {
-               if(fork() == 0) {
-                       if(dpy)
-                               close(ConnectionNumber(dpy));
-                       setsid();
-                       execvp(((char **)arg->v)[0], (char **)arg->v);
-                       fprintf(stderr, "dwm: execvp %s", ((char **)arg->v)[0]);
-                       perror(" failed");
-               }
+               if(dpy)
+                       close(ConnectionNumber(dpy));
+               setsid();
+               execvp(((char **)arg->v)[0], (char **)arg->v);
+               fprintf(stderr, "dwm: execvp %s", ((char **)arg->v)[0]);
+               perror(" failed");
                exit(0);
        }
-       wait(0);
 }
 
 void