Feature: sbd-inquisitor: don't fork if no pid-file given

src/sbd-common.c

@@ -1160,11 +1160,28 @@ sbd_cdtocoredir(void)
 	return rc;
 }
 
+void
+sbd_detach(void)
+{
+    const char *devnull = "/dev/null";
+
+    maximize_priority();
+    sysrq_init();
+
+    umask(022);
+    close(0);
+    (void)open(devnull, O_RDONLY);
+    close(1);
+    (void)open(devnull, O_WRONLY);
+    close(2);
+    (void)open(devnull, O_WRONLY);
+    sbd_cdtocoredir();
+}
+
 pid_t
 make_daemon(void)
 {
 	pid_t			pid;
-	const char *		devnull = "/dev/null";
 
 	pid = fork();
 	if (pid < 0) {
@@ -1179,17 +1196,8 @@ make_daemon(void)
         qb_log_ctl(QB_LOG_STDERR, QB_LOG_CONF_ENABLED, QB_FALSE);
 
 	/* This is the child; ensure privileges have not been lost. */
-	maximize_priority();
-	sysrq_init();
-
-	umask(022);
-	close(0);
-	(void)open(devnull, O_RDONLY);
-	close(1);
-	(void)open(devnull, O_WRONLY);
-	close(2);
-	(void)open(devnull, O_WRONLY);
-	sbd_cdtocoredir();
+	sbd_detach();
+
 	return 0;
 }
 

src/sbd-inquisitor.c

@@ -1387,7 +1387,14 @@ int main(int argc, char **argv, char **envp)
 
                 cl_log(LOG_NOTICE, "%s flush + write \'%c\' to sysrq in case of timeout",
                        do_flush?"Do":"Skip", timeout_sysrq_char);
-                exit_status = inquisitor();
+                if (pidfile) {
+                    exit_status = inquisitor();
+                } else {
+                    sbd_detach();
+                    inquisitor_child();
+                    /* not reached */
+                    exit(0);
+                }
         } else {
             exit_status = -2;
         }

src/sbd.h

@@ -136,6 +136,7 @@ void do_reset(void);
 void do_off(void);
 void do_timeout_action(void);
 pid_t make_daemon(void);
+void sbd_detach(void);
 void maximize_priority(void);
 void sbd_get_uname(void);
 void sbd_set_format_string(int method, const char *daemon);