Make set_pidfile() easier to use.

diff --git a/controller/controller.c b/controller/controller.c
index 8e2c7b5..61a550a 100644 (file)
--- a/controller/controller.c
+++ b/controller/controller.c
@@ -253,7 +253,7 @@ parse_options(int argc, char *argv[])
             break;
 
         case 'P':
-            set_pidfile(optarg ? optarg : "controller.pid");
+            set_pidfile(optarg);
             break;
 
         case 'H':

diff --git a/lib/daemon.c b/lib/daemon.c
index 2a835be..e14b776 100644 (file)
--- a/lib/daemon.c
+++ b/lib/daemon.c
@@ -52,12 +52,16 @@ static char *pidfile;
 /* Sets up a following call to daemonize() to create a pidfile named 'name'.
  * If 'name' begins with '/', then it is treated as an absolute path.
  * Otherwise, it is taken relative to RUNDIR, which is $(prefix)/var/run by
- * default. */
+ * default.
+ *
+ * If 'name' is null, then program_name followed by ".pid" is used. */
 void
 set_pidfile(const char *name)
 {
     free(pidfile);
-    pidfile = *name == '/' ? xstrdup(name) : xasprintf("%s/%s", RUNDIR, name);
+    pidfile = (!name ? xasprintf("%s/%s.pid", RUNDIR, program_name)
+               : *name == '/' ? xstrdup(name)
+               : xasprintf("%s/%s", RUNDIR, name));
 }
 
 /* Sets up a following call to daemonize() to detach from the foreground

diff --git a/secchan/secchan.c b/secchan/secchan.c
index 5577a27..3c96312 100644 (file)
--- a/secchan/secchan.c
+++ b/secchan/secchan.c
@@ -754,7 +754,7 @@ parse_options(int argc, char *argv[])
             break;
 
         case 'P':
-            set_pidfile(optarg ? optarg : "secchan.pid");
+            set_pidfile(optarg);
             break;
 
         case 'l':

diff --git a/switch/switch.c b/switch/switch.c
index ee61fd6..d745bc8 100644 (file)
--- a/switch/switch.c
+++ b/switch/switch.c
@@ -200,7 +200,7 @@ parse_options(int argc, char *argv[])
             break;
 
         case 'P':
-            set_pidfile(optarg ? optarg : "switch.pid");
+            set_pidfile(optarg);
             break;
 
         case 'v':