From 56d4fbf92eb6d8eb31c910e137224308c40dd909 Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Sat, 28 Jul 2012 13:20:35 +0200 Subject: [PATCH 1/1] systemctl: append .service to unit names lacking suffix https://bugs.freedesktop.org/show_bug.cgi?id=39386 --- TODO | 4 ---- src/shared/unit-name.c | 11 +++++++++-- 2 files changed, 9 insertions(+), 6 deletions(-) diff --git a/TODO b/TODO index d4f350292..59ba7a2e1 100644 --- a/TODO +++ b/TODO @@ -49,10 +49,6 @@ Features: * rename "userspace" to "core-os" -* append ".service" to unit names without any suffix (https://bugs.freedesktop.org/show_bug.cgi?id=39386) - -* journalctl: add --priority switch - * systemctl: "Journal has been rotated since unit was started." message is misleading * syscall filter: add knowledge about compat syscalls diff --git a/src/shared/unit-name.c b/src/shared/unit-name.c index e84d995dd..719371879 100644 --- a/src/shared/unit-name.c +++ b/src/shared/unit-name.c @@ -477,6 +477,7 @@ char *unit_dbus_path_from_name(const char *name) { char *unit_name_mangle(const char *name) { char *r, *t; const char *f; + bool dot = false; assert(name); @@ -493,12 +494,15 @@ char *unit_name_mangle(const char *name) { /* We'll only escape the obvious characters here, to play * safe. */ - r = new(char, strlen(name) * 4 + 1); + r = new(char, strlen(name) * 4 + 1 + sizeof(".service")-1); if (!r) return NULL; for (f = name, t = r; *f; f++) { + if (*f == '.') + dot = true; + if (*f == '/') *(t++) = '-'; else if (!strchr("@" VALID_CHARS, *f)) @@ -507,7 +511,10 @@ char *unit_name_mangle(const char *name) { *(t++) = *f; } - *t = 0; + if (!dot) + strcpy(t, ".service"); + else + *t = 0; return r; } -- 2.30.2