(disassoc): Rewrite for POSIX, return status. Caller changed.

Problems with the old code:
* Insufficient error checking.
* It used TIOCNOTTY (obsolete BSDism) to get rid of the controlling
  tty, except for hpux || Rel4, where it attempted to use the POSIX
  way, but screwed up.
* It left file descriptors 0, 1, 2 in a somewhat weird state.

include/prototypes.h

@@ -371,7 +371,7 @@ extern void print_config(FILE * fp);
 extern int roll(int);
 extern int roundavg(double);
 extern int chance(double);
-extern void disassoc(void);
+extern int disassoc(void);
 extern int diffx(int, int);
 extern int diffy(int, int);
 extern int deltax(int, int);

src/lib/gen/Makefile

@@ -40,7 +40,7 @@ OBJS =  chance.o copy.o disassoc.o \
 	ioqueue.o mapdist.o minmax.o numstr.o onearg.o \
 	parse.o plur.o queue.o round.o scthash.o
 
-NTOBJS = chance.obj copy.obj disassoc.obj \
+NTOBJS = chance.obj copy.obj \
 	emp_config.obj getstarg.obj getstring.obj \
 	io.obj ioqueue.obj mapdist.obj minmax.obj \
 	numstr.obj onearg.obj parse.obj plur.obj queue.obj round.obj \

src/lib/gen/disassoc.c

@@ -25,48 +25,54 @@
  *
  *  ---
  *
- *  disassoc.c: Fork and close
+ *  disassoc.c: Boilerplate daemonization code
  * 
  *  Known contributors to this file:
  *     Doug Hay, 1998
+ *     Markus Armbruster, 2005
  */
 
 /*
- * boilerplate daemon code; disassociate from
+ * See W. Richard Stevens: UNIX Network Programming, Vol. 1
- * the current tty by forking, closing all file
- * descriptors, opening slash, and ioctl-ing
- * TIOCNOTTY
  */
 
-#include <sys/types.h>
-#if !defined(_WIN32)
-#include <sys/ioctl.h>
-#include <unistd.h>		/* fork close dup2 */
-#endif
 #include <fcntl.h>
-#include "gen.h"
+#include <sys/types.h>
+#include <unistd.h>
+#include "prototypes.h"
 
-void
+int
 disassoc(void)
 {
-#if !defined(_WIN32)
+    pid_t pid;
     int i;
 
-    if (fork() != 0)
+    if ((pid = fork()) < 0)
-	exit(0);
+	return -1;
-    for (i = 0; i < 2; i++)
+    else if (pid)
-	(void)close(i);
+	_exit(0);		/* parent */
-    (void)open("/", O_RDONLY, 0);
+
-    (void)dup2(0, 1);
+    /* Become session leader of new session, lose controlling tty */
-    (void)dup2(0, 2);
+    if (setsid() < 0)
-#if defined hpux || defined Rel4
+	return -1;
-    setsid();
+
-#else
+    /* Lose session leader status, so we can't acquire a controlling tty */
-    i = open("/dev/tty", O_RDWR, 0);
+    if ((pid = fork()) < 0)
-    if (i > 0) {
+	return -1;
-	(void)ioctl(i, TIOCNOTTY, 0);
+    else if (pid)
-	(void)close(i);
+	_exit(0);		/* parent */
-    }
+    /* Note: no controlling tty, therefore no SIGHUP sent */
-#endif
+
-#endif
+    /* datadir is working directory, that's fine */
+
+    /* We opened a bunch of files already, so just close 0..2 */
+    for (i = 0; i < 3; i++)
+	close(i);
+
+    /* Library code may use 0..2; make sure that's safe */
+    open("/dev/null", O_RDWR);
+    dup(0);
+    dup(0);
+
+    return 0;
 }

src/server/main.c

@@ -235,8 +235,12 @@ main(int argc, char **argv)
     }
     daemonize = 0;
 #else  /* !_WIN32 */
-    if (daemonize)
+    if (daemonize) {
-	disassoc();
+	if (disassoc() < 0) {
+	    logerror("Can't become daemon (%s)", strerror(errno));
+	    _exit(1);
+	}
+    }
 #endif /* !_WIN32 */
     start_server(flags);