Make set_pidfile() easier to use.

diff --git a/controller/controller.c b/controller/controller.c
index 8e2c7b5c802160a07fd0b4017e95bba1c2bb0682..61a550a41c821a2316ef5d8bb7af152d8b765086 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 2a835be9702591c773ca66f6a310bcfc9734c462..e14b776d266c1bd10ee8cba693653459f4537ed1 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 5577a27afb582f665f6e14bcaf8ef88684c0897e..3c96312f53b0eecb547b4002342865016550a474 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 ee61fd6dbdbda7efecd56bd46b424832678233bf..d745bc83566c953304215d5b234601f45410efe6 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':