X-Git-Url: https://jasonwoof.com/gitweb/?p=dwm.git;a=blobdiff_plain;f=util.c;h=abd82defb07cddee6a50117452a15ee94ce19a8f;hp=2eb9eb2420ccec6ae1b5e2da1649b960e588ee91;hb=6651dd7fd9e8e95cfc6c472f1adfeff41735d798;hpb=6a3ae5e26a5cebfd35f2423aec0c47843b5a85be diff --git a/util.c b/util.c index 2eb9eb2..abd82de 100644 --- a/util.c +++ b/util.c @@ -1,5 +1,4 @@ -/* - * (C)opyright MMVI Anselm R. Garbe +/* (C)opyright MMVI Anselm R. Garbe * See LICENSE file for license details. */ #include "dwm.h" @@ -12,8 +11,7 @@ /* extern */ void * -emallocz(unsigned int size) -{ +emallocz(unsigned int size) { void *res = calloc(1, size); if(!res) @@ -22,8 +20,7 @@ emallocz(unsigned int size) } void -eprint(const char *errstr, ...) -{ +eprint(const char *errstr, ...) { va_list ap; va_start(ap, errstr); @@ -33,30 +30,30 @@ eprint(const char *errstr, ...) } void * -erealloc(void *ptr, unsigned int size) -{ +erealloc(void *ptr, unsigned int size) { void *res = realloc(ptr, size); + if(!res) eprint("fatal: could not malloc() %u bytes\n", size); return res; } void -spawn(Arg *arg) -{ +spawn(Arg *arg) { static char *shell = NULL; if(!shell && !(shell = getenv("SHELL"))) shell = "/bin/sh"; - if(!arg->cmd) return; + /* The double-fork construct avoids zombie processes and keeps the code + * clean from stupid signal handlers. */ if(fork() == 0) { if(fork() == 0) { if(dpy) close(ConnectionNumber(dpy)); setsid(); - execl(shell, shell, "-c", arg->cmd, NULL); + execl(shell, shell, "-c", arg->cmd, (char *)NULL); fprintf(stderr, "dwm: execl '%s -c %s'", shell, arg->cmd); perror(" failed"); }