One more comment about this patch... On Sat, 2014-10-18 at 20:43 +0200, Colin Guthrie wrote: > @@ -374,7 +378,7 @@ int main(int argc, char *argv[]) { > int r = 0, retval = 1, d = 0; > bool valid_pid_file = false; > bool ltdl_init = false; > - int passed_fd = -1; > + int n, *passed_fds = NULL; n is only used if HAVE_SYSTEMD_DAEMON is defined. If it's not defined, there will be a compiler warning about unused variable. -- Tanu