cryptsetup: fix escaping when generating cryptsetup units
This commit is contained in:
parent
61beaf818b
commit
35eb6b124e
4
TODO
4
TODO
|
@ -25,6 +25,10 @@ Bugfixes:
|
||||||
|
|
||||||
Features:
|
Features:
|
||||||
|
|
||||||
|
* add switch to journalctl to only show data from current boot
|
||||||
|
|
||||||
|
* change REquires=basic.target to RequisiteOverride=basic.target
|
||||||
|
|
||||||
* turn $NOTIFY_SOCKET back into an abstract namespace socket for
|
* turn $NOTIFY_SOCKET back into an abstract namespace socket for
|
||||||
compatibility with services which chroot()
|
compatibility with services which chroot()
|
||||||
|
|
||||||
|
|
|
@ -2509,7 +2509,8 @@ int unit_add_node_link(Unit *u, const char *what, bool wants) {
|
||||||
if (!is_device_path(what))
|
if (!is_device_path(what))
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
if (!(e = unit_name_build_escape(what+1, NULL, ".device")))
|
e = unit_name_from_path(what, ".device");
|
||||||
|
if (!e)
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
|
||||||
r = manager_load_unit(u->manager, e, NULL, NULL, &device);
|
r = manager_load_unit(u->manager, e, NULL, NULL, &device);
|
||||||
|
|
|
@ -81,7 +81,7 @@ static int create_disk(
|
||||||
noauto = has_option(options, "noauto");
|
noauto = has_option(options, "noauto");
|
||||||
nofail = has_option(options, "nofail");
|
nofail = has_option(options, "nofail");
|
||||||
|
|
||||||
n = unit_name_build_escape("systemd-cryptsetup", name, ".service");
|
n = unit_name_from_path_instance("systemd-cryptsetup", name, ".service");
|
||||||
if (!n) {
|
if (!n) {
|
||||||
r = -ENOMEM;
|
r = -ENOMEM;
|
||||||
log_error("Failed to allocate unit name.");
|
log_error("Failed to allocate unit name.");
|
||||||
|
|
|
@ -112,7 +112,8 @@ int unit_name_to_instance(const char *n, char **instance) {
|
||||||
assert(instance);
|
assert(instance);
|
||||||
|
|
||||||
/* Everything past the first @ and before the last . is the instance */
|
/* Everything past the first @ and before the last . is the instance */
|
||||||
if (!(p = strchr(n, '@'))) {
|
p = strchr(n, '@');
|
||||||
|
if (!p) {
|
||||||
*instance = NULL;
|
*instance = NULL;
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
@ -120,7 +121,8 @@ int unit_name_to_instance(const char *n, char **instance) {
|
||||||
assert_se(d = strrchr(n, '.'));
|
assert_se(d = strrchr(n, '.'));
|
||||||
assert(p < d);
|
assert(p < d);
|
||||||
|
|
||||||
if (!(i = strndup(p+1, d-p-1)))
|
i = strndup(p+1, d-p-1);
|
||||||
|
if (!i)
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
|
||||||
*instance = i;
|
*instance = i;
|
||||||
|
@ -211,50 +213,6 @@ static char *do_escape(const char *f, char *t) {
|
||||||
return t;
|
return t;
|
||||||
}
|
}
|
||||||
|
|
||||||
char *unit_name_build_escape(const char *prefix, const char *instance, const char *suffix) {
|
|
||||||
char *r, *t;
|
|
||||||
size_t a, b, c;
|
|
||||||
|
|
||||||
assert(prefix);
|
|
||||||
assert(suffix);
|
|
||||||
|
|
||||||
/* Takes a arbitrary string for prefix and instance plus a
|
|
||||||
* suffix and makes a nice string suitable as unit name of it,
|
|
||||||
* escaping all weird chars on the way.
|
|
||||||
*
|
|
||||||
* / becomes -, and all chars not allowed in a unit name get
|
|
||||||
* escaped as \xFF, including \ and -, of course. This
|
|
||||||
* escaping is hence reversible.
|
|
||||||
*
|
|
||||||
* This is primarily useful to make nice unit names from
|
|
||||||
* strings, but is actually useful for any kind of string.
|
|
||||||
*/
|
|
||||||
|
|
||||||
a = strlen(prefix);
|
|
||||||
c = strlen(suffix);
|
|
||||||
|
|
||||||
if (instance) {
|
|
||||||
b = strlen(instance);
|
|
||||||
|
|
||||||
r = new(char, a*4 + 1 + b*4 + c + 1);
|
|
||||||
if (!r)
|
|
||||||
return NULL;
|
|
||||||
|
|
||||||
t = do_escape(prefix, r);
|
|
||||||
*(t++) = '@';
|
|
||||||
t = do_escape(instance, t);
|
|
||||||
} else {
|
|
||||||
|
|
||||||
if (!(r = new(char, a*4 + c + 1)))
|
|
||||||
return NULL;
|
|
||||||
|
|
||||||
t = do_escape(prefix, r);
|
|
||||||
}
|
|
||||||
|
|
||||||
strcpy(t, suffix);
|
|
||||||
return r;
|
|
||||||
}
|
|
||||||
|
|
||||||
char *unit_name_escape(const char *f) {
|
char *unit_name_escape(const char *f) {
|
||||||
char *r, *t;
|
char *r, *t;
|
||||||
|
|
||||||
|
@ -301,6 +259,49 @@ char *unit_name_unescape(const char *f) {
|
||||||
return r;
|
return r;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
char *unit_name_path_escape(const char *f) {
|
||||||
|
char *p, *e;
|
||||||
|
|
||||||
|
assert(f);
|
||||||
|
|
||||||
|
p = strdup(f);
|
||||||
|
if (!p)
|
||||||
|
return NULL;
|
||||||
|
|
||||||
|
path_kill_slashes(p);
|
||||||
|
|
||||||
|
if (streq(p, "/")) {
|
||||||
|
free(p);
|
||||||
|
return strdup("-");
|
||||||
|
}
|
||||||
|
|
||||||
|
e = unit_name_escape(p[0] == '/' ? p + 1 : p);
|
||||||
|
free(p);
|
||||||
|
|
||||||
|
return e;
|
||||||
|
}
|
||||||
|
|
||||||
|
char *unit_name_path_unescape(const char *f) {
|
||||||
|
char *e;
|
||||||
|
|
||||||
|
assert(f);
|
||||||
|
|
||||||
|
e = unit_name_unescape(f);
|
||||||
|
if (!e)
|
||||||
|
return NULL;
|
||||||
|
|
||||||
|
if (e[0] != '/') {
|
||||||
|
char *w;
|
||||||
|
|
||||||
|
w = strappend("/", e);
|
||||||
|
free(e);
|
||||||
|
|
||||||
|
return w;
|
||||||
|
}
|
||||||
|
|
||||||
|
return e;
|
||||||
|
}
|
||||||
|
|
||||||
bool unit_name_is_template(const char *n) {
|
bool unit_name_is_template(const char *n) {
|
||||||
const char *p;
|
const char *p;
|
||||||
|
|
||||||
|
@ -329,14 +330,16 @@ char *unit_name_replace_instance(const char *f, const char *i) {
|
||||||
|
|
||||||
b = strlen(i);
|
b = strlen(i);
|
||||||
|
|
||||||
if (!(r = new(char, a + 1 + b + strlen(e) + 1)))
|
r = new(char, a + 1 + b + strlen(e) + 1);
|
||||||
|
if (!r)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
k = mempcpy(r, f, a + 1);
|
k = mempcpy(r, f, a + 1);
|
||||||
k = mempcpy(k, i, b);
|
k = mempcpy(k, i, b);
|
||||||
} else {
|
} else {
|
||||||
|
|
||||||
if (!(r = new(char, a + strlen(e) + 1)))
|
r = new(char, a + strlen(e) + 1);
|
||||||
|
if (!r)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
k = mempcpy(r, f, a);
|
k = mempcpy(r, f, a);
|
||||||
|
@ -373,20 +376,11 @@ char *unit_name_from_path(const char *path, const char *suffix) {
|
||||||
assert(path);
|
assert(path);
|
||||||
assert(suffix);
|
assert(suffix);
|
||||||
|
|
||||||
p = strdup(path);
|
p = unit_name_path_escape(path);
|
||||||
if (!p)
|
if (!p)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
path_kill_slashes(p);
|
r = strappend(p, suffix);
|
||||||
|
|
||||||
path = p[0] == '/' ? p + 1 : p;
|
|
||||||
|
|
||||||
if (path[0] == 0) {
|
|
||||||
free(p);
|
|
||||||
return strappend("-", suffix);
|
|
||||||
}
|
|
||||||
|
|
||||||
r = unit_name_build_escape(path, NULL, suffix);
|
|
||||||
free(p);
|
free(p);
|
||||||
|
|
||||||
return r;
|
return r;
|
||||||
|
@ -395,22 +389,15 @@ char *unit_name_from_path(const char *path, const char *suffix) {
|
||||||
char *unit_name_from_path_instance(const char *prefix, const char *path, const char *suffix) {
|
char *unit_name_from_path_instance(const char *prefix, const char *path, const char *suffix) {
|
||||||
char *p, *r;
|
char *p, *r;
|
||||||
|
|
||||||
|
assert(prefix);
|
||||||
assert(path);
|
assert(path);
|
||||||
assert(suffix);
|
assert(suffix);
|
||||||
|
|
||||||
if (!(p = strdup(path)))
|
p = unit_name_path_escape(path);
|
||||||
|
if (!p)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
path_kill_slashes(p);
|
r = join(prefix, "@", p, suffix, NULL);
|
||||||
|
|
||||||
path = p[0] == '/' ? p + 1 : p;
|
|
||||||
|
|
||||||
if (path[0] == 0) {
|
|
||||||
free(p);
|
|
||||||
return unit_name_build_escape(prefix, "-", suffix);
|
|
||||||
}
|
|
||||||
|
|
||||||
r = unit_name_build_escape(prefix, path, suffix);
|
|
||||||
free(p);
|
free(p);
|
||||||
|
|
||||||
return r;
|
return r;
|
||||||
|
@ -425,52 +412,17 @@ char *unit_name_to_path(const char *name) {
|
||||||
if (!w)
|
if (!w)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
e = unit_name_unescape(w);
|
e = unit_name_path_unescape(w);
|
||||||
free(w);
|
free(w);
|
||||||
|
|
||||||
if (!e)
|
|
||||||
return NULL;
|
|
||||||
|
|
||||||
if (e[0] != '/') {
|
|
||||||
w = strappend("/", e);
|
|
||||||
free(e);
|
|
||||||
|
|
||||||
if (!w)
|
|
||||||
return NULL;
|
|
||||||
|
|
||||||
return w;
|
|
||||||
}
|
|
||||||
|
|
||||||
return e;
|
|
||||||
}
|
|
||||||
|
|
||||||
char *unit_name_path_unescape(const char *f) {
|
|
||||||
char *e;
|
|
||||||
|
|
||||||
assert(f);
|
|
||||||
|
|
||||||
e = unit_name_unescape(f);
|
|
||||||
if (!e)
|
|
||||||
return NULL;
|
|
||||||
|
|
||||||
if (e[0] != '/') {
|
|
||||||
char *w;
|
|
||||||
|
|
||||||
w = strappend("/", e);
|
|
||||||
free(e);
|
|
||||||
|
|
||||||
if (!w)
|
|
||||||
return NULL;
|
|
||||||
|
|
||||||
return w;
|
|
||||||
}
|
|
||||||
|
|
||||||
return e;
|
return e;
|
||||||
}
|
}
|
||||||
|
|
||||||
char *unit_dbus_path_from_name(const char *name) {
|
char *unit_dbus_path_from_name(const char *name) {
|
||||||
char *e, *p;
|
char *e, *p;
|
||||||
|
|
||||||
|
assert(name);
|
||||||
|
|
||||||
e = bus_path_escape(name);
|
e = bus_path_escape(name);
|
||||||
if (!e)
|
if (!e)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
|
@ -37,11 +37,10 @@ bool unit_instance_is_valid(const char *i);
|
||||||
char *unit_name_change_suffix(const char *n, const char *suffix);
|
char *unit_name_change_suffix(const char *n, const char *suffix);
|
||||||
|
|
||||||
char *unit_name_build(const char *prefix, const char *instance, const char *suffix);
|
char *unit_name_build(const char *prefix, const char *instance, const char *suffix);
|
||||||
char *unit_name_build_escape(const char *prefix, const char *instance, const char *suffix);
|
|
||||||
|
|
||||||
char *unit_name_escape(const char *f);
|
char *unit_name_escape(const char *f);
|
||||||
char *unit_name_unescape(const char *f);
|
char *unit_name_unescape(const char *f);
|
||||||
|
char *unit_name_path_escape(const char *f);
|
||||||
char *unit_name_path_unescape(const char *f);
|
char *unit_name_path_unescape(const char *f);
|
||||||
|
|
||||||
bool unit_name_is_template(const char *n);
|
bool unit_name_is_template(const char *n);
|
||||||
|
|
|
@ -29,6 +29,57 @@
|
||||||
int main(int argc, char* argv[]) {
|
int main(int argc, char* argv[]) {
|
||||||
char *t, *k;
|
char *t, *k;
|
||||||
|
|
||||||
|
t = unit_name_from_path("/waldo", ".mount");
|
||||||
|
puts(t);
|
||||||
|
k = unit_name_to_path(t);
|
||||||
|
puts(k);
|
||||||
|
free(k);
|
||||||
|
free(t);
|
||||||
|
|
||||||
|
t = unit_name_from_path("/waldo/quuix", ".mount");
|
||||||
|
puts(t);
|
||||||
|
k = unit_name_to_path(t);
|
||||||
|
puts(k);
|
||||||
|
free(k);
|
||||||
|
free(t);
|
||||||
|
|
||||||
|
t = unit_name_from_path("/waldo/quuix/", ".mount");
|
||||||
|
puts(t);
|
||||||
|
k = unit_name_to_path(t);
|
||||||
|
puts(k);
|
||||||
|
free(k);
|
||||||
|
free(t);
|
||||||
|
|
||||||
|
t = unit_name_from_path("/", ".mount");
|
||||||
|
puts(t);
|
||||||
|
k = unit_name_to_path(t);
|
||||||
|
puts(k);
|
||||||
|
free(k);
|
||||||
|
free(t);
|
||||||
|
|
||||||
|
t = unit_name_from_path("///", ".mount");
|
||||||
|
puts(t);
|
||||||
|
k = unit_name_to_path(t);
|
||||||
|
puts(k);
|
||||||
|
free(k);
|
||||||
|
free(t);
|
||||||
|
|
||||||
|
t = unit_name_from_path_instance("waldo", "/waldo", ".mount");
|
||||||
|
puts(t);
|
||||||
|
free(t);
|
||||||
|
|
||||||
|
t = unit_name_from_path_instance("waldo", "/waldo////quuix////", ".mount");
|
||||||
|
puts(t);
|
||||||
|
free(t);
|
||||||
|
|
||||||
|
t = unit_name_from_path_instance("waldo", "/", ".mount");
|
||||||
|
puts(t);
|
||||||
|
free(t);
|
||||||
|
|
||||||
|
t = unit_name_from_path_instance("wa--ldo", "/--", ".mount");
|
||||||
|
puts(t);
|
||||||
|
free(t);
|
||||||
|
|
||||||
assert_se(t = unit_name_mangle("/home"));
|
assert_se(t = unit_name_mangle("/home"));
|
||||||
assert_se(k = unit_name_mangle(t));
|
assert_se(k = unit_name_mangle(t));
|
||||||
puts(t);
|
puts(t);
|
||||||
|
|
Loading…
Reference in New Issue