unit-name: rework unit_name_replace_instance function()

https://bugzilla.redhat.com/show_bug.cgi?id=855863

diff --git a/TODO b/TODO
index 4371c730a737a69ed63fcf56c92329d598b8bce3..29170685b1cf339a7ff3bddca9f30ac7059d58d7 100644 (file)
--- a/TODO
+++ b/TODO
@@ -49,6 +49,18 @@ Bugfixes:
 
 Features:
 
 
 Features:
 

+* perfomance messages for selinux are gone from debug log?
+
+* http://lists.freedesktop.org/archives/systemd-devel/2012-September/006502.html
+
+* don't use writev() in tmpfiles for sake of compat with sysfs?
+
+* come up with a nice way to write queue/read_ahead_kb for a block device without interfering with readahead
+
+* journald: add kernel cmdline option to disable ratelimiting for debug purposes
+
+* Add a way to reference the machine/boot ID from ExecStart= and similar command lines
+

 * move PID 1 segfaults to /var/lib/systemd/coredump?
 
 * Document word splitting syntax for ExecStart= and friends
 * move PID 1 segfaults to /var/lib/systemd/coredump?
 
 * Document word splitting syntax for ExecStart= and friends

diff --git a/src/shared/unit-name.c b/src/shared/unit-name.c
index 71937187922835fb3fcf3d6c3c8037356f0bfa9d..d6391228f2118686d4cae9cd79ca230a3d8ffc20 100644 (file)
--- a/src/shared/unit-name.c
+++ b/src/shared/unit-name.c
@@ -357,36 +357,29 @@ bool unit_name_is_instance(const char *n) {
 char *unit_name_replace_instance(const char *f, const char *i) {
         const char *p, *e;
         char *r, *k;
 char *unit_name_replace_instance(const char *f, const char *i) {
         const char *p, *e;
         char *r, *k;

-        size_t a;
+        size_t a, b;

 
         assert(f);
 
         p = strchr(f, '@');
 
         assert(f);
 
         p = strchr(f, '@');

-        assert_se(e = strrchr(f, '.'));
-
-        a = p - f;
-
-        if (p) {
-                size_t b;
-
-                b = strlen(i);
-
-                r = new(char, a + 1 + b + strlen(e) + 1);
-                if (!r)
-                        return NULL;
+        if (!p)
+                return strdup(f);

 
 

-                k = mempcpy(r, f, a + 1);
-                k = mempcpy(k, i, b);
-        } else {
+        e = strrchr(f, '.');
+        if (!e)
+                assert_se(e = strchr(f, 0));

 
 

-                r = new(char, a + strlen(e) + 1);
-                if (!r)
-                        return NULL;
+        a = p - f;
+        b = strlen(i);

 
 

-                k = mempcpy(r, f, a);
-        }
+        r = new(char, a + 1 + b + strlen(e) + 1);
+        if (!r)
+                return NULL;

 
 

+        k = mempcpy(r, f, a + 1);
+        k = mempcpy(k, i, b);

         strcpy(k, e);
         strcpy(k, e);

+

         return r;
 }
 
         return r;
 }
 

diff --git a/src/test/test-unit-name.c b/src/test/test-unit-name.c
index 392e358ac2f4efc7cccfda0aee17a90ea6ff55bf..cf434558c34f4ced62df24bd961242794c816f68 100644 (file)
--- a/src/test/test-unit-name.c
+++ b/src/test/test-unit-name.c
@@ -29,6 +29,30 @@
 int main(int argc, char* argv[]) {
         char *t, *k;
 
 int main(int argc, char* argv[]) {
         char *t, *k;
 

+        t = unit_name_replace_instance("foo@.service", "waldo");
+        puts(t);
+        free(t);
+
+        t = unit_name_replace_instance("foo@xyz.service", "waldo");
+        puts(t);
+        free(t);
+
+        t = unit_name_replace_instance("xyz", "waldo");
+        puts(t);
+        free(t);
+
+        t = unit_name_replace_instance("", "waldo");
+        puts(t);
+        free(t);
+
+        t = unit_name_replace_instance("", "");
+        puts(t);
+        free(t);
+
+        t = unit_name_replace_instance("foo.service", "waldo");
+        puts(t);
+        free(t);
+

         t = unit_name_from_path("/waldo", ".mount");
         puts(t);
         k = unit_name_to_path(t);
         t = unit_name_from_path("/waldo", ".mount");
         puts(t);
         k = unit_name_to_path(t);