|
|
1854ec0 |
From 40c5012fb52b8e9fe19da18a4f0ca2282e780fed Mon Sep 17 00:00:00 2001
|
|
|
1854ec0 |
From: Lennart Poettering <lennart@poettering.net>
|
|
|
1854ec0 |
Date: Tue, 10 Jul 2012 17:03:11 +0200
|
|
|
1854ec0 |
Subject: [PATCH] unit-name: remove unit_name_is_valid_no_type() and move
|
|
|
1854ec0 |
unit_name_is_valid() to unit-name.h (cherry picked from
|
|
|
1854ec0 |
commit 5f73969991fa765f2826975c0fc5e47438b5e9ea)
|
|
|
1854ec0 |
|
|
|
1854ec0 |
---
|
|
|
1854ec0 |
src/core/manager.c | 2 +-
|
|
|
1854ec0 |
src/core/unit.c | 22 ----------------------
|
|
|
1854ec0 |
src/core/unit.h | 3 ---
|
|
|
1854ec0 |
src/shared/install.c | 18 +++++++++---------
|
|
|
1854ec0 |
src/shared/unit-name.c | 19 +++++++++++++++++--
|
|
|
1854ec0 |
src/shared/unit-name.h | 4 +++-
|
|
|
1854ec0 |
6 files changed, 30 insertions(+), 38 deletions(-)
|
|
|
1854ec0 |
|
|
|
1854ec0 |
diff --git a/src/core/manager.c b/src/core/manager.c
|
|
|
1854ec0 |
index 2a17794..64c7823 100644
|
|
|
1854ec0 |
--- a/src/core/manager.c
|
|
|
1854ec0 |
+++ b/src/core/manager.c
|
|
|
1854ec0 |
@@ -797,7 +797,7 @@ int manager_load_unit_prepare(Manager *m, const char *name, const char *path, DB
|
|
|
1854ec0 |
|
|
|
1854ec0 |
t = unit_name_to_type(name);
|
|
|
1854ec0 |
|
|
|
1854ec0 |
- if (t == _UNIT_TYPE_INVALID || !unit_name_is_valid_no_type(name, false)) {
|
|
|
1854ec0 |
+ if (t == _UNIT_TYPE_INVALID || !unit_name_is_valid(name, false)) {
|
|
|
1854ec0 |
dbus_set_error(e, BUS_ERROR_INVALID_NAME, "Unit name %s is not valid.", name);
|
|
|
1854ec0 |
return -EINVAL;
|
|
|
1854ec0 |
}
|
|
|
1854ec0 |
diff --git a/src/core/unit.c b/src/core/unit.c
|
|
|
1854ec0 |
index b52ed42..f5ced75 100644
|
|
|
1854ec0 |
--- a/src/core/unit.c
|
|
|
1854ec0 |
+++ b/src/core/unit.c
|
|
|
1854ec0 |
@@ -2635,28 +2635,6 @@ bool unit_pending_active(Unit *u) {
|
|
|
1854ec0 |
return false;
|
|
|
1854ec0 |
}
|
|
|
1854ec0 |
|
|
|
1854ec0 |
-UnitType unit_name_to_type(const char *n) {
|
|
|
1854ec0 |
- UnitType t;
|
|
|
1854ec0 |
-
|
|
|
1854ec0 |
- assert(n);
|
|
|
1854ec0 |
-
|
|
|
1854ec0 |
- for (t = 0; t < _UNIT_TYPE_MAX; t++)
|
|
|
1854ec0 |
- if (endswith(n, unit_vtable[t]->suffix))
|
|
|
1854ec0 |
- return t;
|
|
|
1854ec0 |
-
|
|
|
1854ec0 |
- return _UNIT_TYPE_INVALID;
|
|
|
1854ec0 |
-}
|
|
|
1854ec0 |
-
|
|
|
1854ec0 |
-bool unit_name_is_valid(const char *n, bool template_ok) {
|
|
|
1854ec0 |
- UnitType t;
|
|
|
1854ec0 |
-
|
|
|
1854ec0 |
- t = unit_name_to_type(n);
|
|
|
1854ec0 |
- if (t < 0 || t >= _UNIT_TYPE_MAX)
|
|
|
1854ec0 |
- return false;
|
|
|
1854ec0 |
-
|
|
|
1854ec0 |
- return unit_name_is_valid_no_type(n, template_ok);
|
|
|
1854ec0 |
-}
|
|
|
1854ec0 |
-
|
|
|
1854ec0 |
int unit_kill(Unit *u, KillWho w, KillMode m, int signo, DBusError *error) {
|
|
|
1854ec0 |
assert(u);
|
|
|
1854ec0 |
assert(w >= 0 && w < _KILL_WHO_MAX);
|
|
|
1854ec0 |
diff --git a/src/core/unit.h b/src/core/unit.h
|
|
|
1854ec0 |
index c994107..267a03e 100644
|
|
|
1854ec0 |
--- a/src/core/unit.h
|
|
|
1854ec0 |
+++ b/src/core/unit.h
|
|
|
1854ec0 |
@@ -538,9 +538,6 @@ int unit_add_default_target_dependency(Unit *u, Unit *target);
|
|
|
1854ec0 |
|
|
|
1854ec0 |
int unit_following_set(Unit *u, Set **s);
|
|
|
1854ec0 |
|
|
|
1854ec0 |
-UnitType unit_name_to_type(const char *n);
|
|
|
1854ec0 |
-bool unit_name_is_valid(const char *n, bool template_ok);
|
|
|
1854ec0 |
-
|
|
|
1854ec0 |
void unit_trigger_on_failure(Unit *u);
|
|
|
1854ec0 |
|
|
|
1854ec0 |
bool unit_condition_test(Unit *u);
|
|
|
1854ec0 |
diff --git a/src/shared/install.c b/src/shared/install.c
|
|
|
1854ec0 |
index f443578..786846e 100644
|
|
|
1854ec0 |
--- a/src/shared/install.c
|
|
|
1854ec0 |
+++ b/src/shared/install.c
|
|
|
1854ec0 |
@@ -607,7 +607,7 @@ int unit_file_mask(
|
|
|
1854ec0 |
STRV_FOREACH(i, files) {
|
|
|
1854ec0 |
char *path;
|
|
|
1854ec0 |
|
|
|
1854ec0 |
- if (!unit_name_is_valid_no_type(*i, true)) {
|
|
|
1854ec0 |
+ if (!unit_name_is_valid(*i, true)) {
|
|
|
1854ec0 |
if (r == 0)
|
|
|
1854ec0 |
r = -EINVAL;
|
|
|
1854ec0 |
continue;
|
|
|
1854ec0 |
@@ -683,7 +683,7 @@ int unit_file_unmask(
|
|
|
1854ec0 |
STRV_FOREACH(i, files) {
|
|
|
1854ec0 |
char *path;
|
|
|
1854ec0 |
|
|
|
1854ec0 |
- if (!unit_name_is_valid_no_type(*i, true)) {
|
|
|
1854ec0 |
+ if (!unit_name_is_valid(*i, true)) {
|
|
|
1854ec0 |
if (r == 0)
|
|
|
1854ec0 |
r = -EINVAL;
|
|
|
1854ec0 |
continue;
|
|
|
1854ec0 |
@@ -759,7 +759,7 @@ int unit_file_link(
|
|
|
1854ec0 |
fn = path_get_file_name(*i);
|
|
|
1854ec0 |
|
|
|
1854ec0 |
if (!path_is_absolute(*i) ||
|
|
|
1854ec0 |
- !unit_name_is_valid_no_type(fn, true)) {
|
|
|
1854ec0 |
+ !unit_name_is_valid(fn, true)) {
|
|
|
1854ec0 |
if (r == 0)
|
|
|
1854ec0 |
r = -EINVAL;
|
|
|
1854ec0 |
continue;
|
|
|
1854ec0 |
@@ -922,7 +922,7 @@ static int install_info_add(
|
|
|
1854ec0 |
if (!name)
|
|
|
1854ec0 |
name = path_get_file_name(path);
|
|
|
1854ec0 |
|
|
|
1854ec0 |
- if (!unit_name_is_valid_no_type(name, true))
|
|
|
1854ec0 |
+ if (!unit_name_is_valid(name, true))
|
|
|
1854ec0 |
return -EINVAL;
|
|
|
1854ec0 |
|
|
|
1854ec0 |
if (hashmap_get(c->have_installed, name) ||
|
|
|
1854ec0 |
@@ -1232,7 +1232,7 @@ static int install_info_symlink_wants(
|
|
|
1854ec0 |
STRV_FOREACH(s, i->wanted_by) {
|
|
|
1854ec0 |
char *path;
|
|
|
1854ec0 |
|
|
|
1854ec0 |
- if (!unit_name_is_valid_no_type(*s, true)) {
|
|
|
1854ec0 |
+ if (!unit_name_is_valid(*s, true)) {
|
|
|
1854ec0 |
r = -EINVAL;
|
|
|
1854ec0 |
continue;
|
|
|
1854ec0 |
}
|
|
|
1854ec0 |
@@ -1266,7 +1266,7 @@ static int install_info_symlink_requires(
|
|
|
1854ec0 |
STRV_FOREACH(s, i->required_by) {
|
|
|
1854ec0 |
char *path;
|
|
|
1854ec0 |
|
|
|
1854ec0 |
- if (!unit_name_is_valid_no_type(*s, true)) {
|
|
|
1854ec0 |
+ if (!unit_name_is_valid(*s, true)) {
|
|
|
1854ec0 |
r = -EINVAL;
|
|
|
1854ec0 |
continue;
|
|
|
1854ec0 |
}
|
|
|
1854ec0 |
@@ -1597,7 +1597,7 @@ UnitFileState unit_file_get_state(
|
|
|
1854ec0 |
if (root_dir && scope != UNIT_FILE_SYSTEM)
|
|
|
1854ec0 |
return -EINVAL;
|
|
|
1854ec0 |
|
|
|
1854ec0 |
- if (!unit_name_is_valid_no_type(name, true))
|
|
|
1854ec0 |
+ if (!unit_name_is_valid(name, true))
|
|
|
1854ec0 |
return -EINVAL;
|
|
|
1854ec0 |
|
|
|
1854ec0 |
r = lookup_paths_init_from_scope(&paths, scope);
|
|
|
1854ec0 |
@@ -1792,7 +1792,7 @@ int unit_file_preset(
|
|
|
1854ec0 |
|
|
|
1854ec0 |
STRV_FOREACH(i, files) {
|
|
|
1854ec0 |
|
|
|
1854ec0 |
- if (!unit_name_is_valid_no_type(*i, true)) {
|
|
|
1854ec0 |
+ if (!unit_name_is_valid(*i, true)) {
|
|
|
1854ec0 |
r = -EINVAL;
|
|
|
1854ec0 |
goto finish;
|
|
|
1854ec0 |
}
|
|
|
1854ec0 |
@@ -1897,7 +1897,7 @@ int unit_file_get_list(
|
|
|
1854ec0 |
if (ignore_file(de->d_name))
|
|
|
1854ec0 |
continue;
|
|
|
1854ec0 |
|
|
|
1854ec0 |
- if (!unit_name_is_valid_no_type(de->d_name, true))
|
|
|
1854ec0 |
+ if (!unit_name_is_valid(de->d_name, true))
|
|
|
1854ec0 |
continue;
|
|
|
1854ec0 |
|
|
|
1854ec0 |
if (hashmap_get(h, de->d_name))
|
|
|
1854ec0 |
diff --git a/src/shared/unit-name.c b/src/shared/unit-name.c
|
|
|
1854ec0 |
index b962634..d26340e 100644
|
|
|
1854ec0 |
--- a/src/shared/unit-name.c
|
|
|
1854ec0 |
+++ b/src/shared/unit-name.c
|
|
|
1854ec0 |
@@ -48,7 +48,7 @@ static const char* const unit_type_table[_UNIT_TYPE_MAX] = {
|
|
|
1854ec0 |
|
|
|
1854ec0 |
DEFINE_STRING_TABLE_LOOKUP(unit_type, UnitType);
|
|
|
1854ec0 |
|
|
|
1854ec0 |
-bool unit_name_is_valid_no_type(const char *n, bool template_ok) {
|
|
|
1854ec0 |
+bool unit_name_is_valid(const char *n, bool template_ok) {
|
|
|
1854ec0 |
const char *e, *i, *at;
|
|
|
1854ec0 |
|
|
|
1854ec0 |
/* Valid formats:
|
|
|
1854ec0 |
@@ -66,6 +66,9 @@ bool unit_name_is_valid_no_type(const char *n, bool template_ok) {
|
|
|
1854ec0 |
if (!e || e == n)
|
|
|
1854ec0 |
return false;
|
|
|
1854ec0 |
|
|
|
1854ec0 |
+ if (unit_type_from_string(e + 1) < 0)
|
|
|
1854ec0 |
+ return false;
|
|
|
1854ec0 |
+
|
|
|
1854ec0 |
for (i = n, at = NULL; i < e; i++) {
|
|
|
1854ec0 |
|
|
|
1854ec0 |
if (*i == '@' && !at)
|
|
|
1854ec0 |
@@ -166,7 +169,7 @@ char *unit_name_change_suffix(const char *n, const char *suffix) {
|
|
|
1854ec0 |
size_t a, b;
|
|
|
1854ec0 |
|
|
|
1854ec0 |
assert(n);
|
|
|
1854ec0 |
- assert(unit_name_is_valid_no_type(n, true));
|
|
|
1854ec0 |
+ assert(unit_name_is_valid(n, true));
|
|
|
1854ec0 |
assert(suffix);
|
|
|
1854ec0 |
|
|
|
1854ec0 |
assert_se(e = strrchr(n, '.'));
|
|
|
1854ec0 |
@@ -486,3 +489,15 @@ char *unit_dbus_path_from_name(const char *name) {
|
|
|
1854ec0 |
|
|
|
1854ec0 |
return p;
|
|
|
1854ec0 |
}
|
|
|
1854ec0 |
+
|
|
|
1854ec0 |
+UnitType unit_name_to_type(const char *n) {
|
|
|
1854ec0 |
+ const char *e;
|
|
|
1854ec0 |
+
|
|
|
1854ec0 |
+ assert(n);
|
|
|
1854ec0 |
+
|
|
|
1854ec0 |
+ e = strrchr(n, '.');
|
|
|
1854ec0 |
+ if (!e)
|
|
|
1854ec0 |
+ return _UNIT_TYPE_INVALID;
|
|
|
1854ec0 |
+
|
|
|
1854ec0 |
+ return unit_type_from_string(e + 1);
|
|
|
1854ec0 |
+}
|
|
|
1854ec0 |
diff --git a/src/shared/unit-name.h b/src/shared/unit-name.h
|
|
|
1854ec0 |
index 6bfba24..80efa00 100644
|
|
|
1854ec0 |
--- a/src/shared/unit-name.h
|
|
|
1854ec0 |
+++ b/src/shared/unit-name.h
|
|
|
1854ec0 |
@@ -50,10 +50,12 @@ int unit_name_to_instance(const char *n, char **instance);
|
|
|
1854ec0 |
char* unit_name_to_prefix(const char *n);
|
|
|
1854ec0 |
char* unit_name_to_prefix_and_instance(const char *n);
|
|
|
1854ec0 |
|
|
|
1854ec0 |
-bool unit_name_is_valid_no_type(const char *n, bool template_ok);
|
|
|
1854ec0 |
+bool unit_name_is_valid(const char *n, bool template_ok);
|
|
|
1854ec0 |
bool unit_prefix_is_valid(const char *p);
|
|
|
1854ec0 |
bool unit_instance_is_valid(const char *i);
|
|
|
1854ec0 |
|
|
|
1854ec0 |
+UnitType unit_name_to_type(const char *n);
|
|
|
1854ec0 |
+
|
|
|
1854ec0 |
char *unit_name_change_suffix(const char *n, const char *suffix);
|
|
|
1854ec0 |
|
|
|
1854ec0 |
char *unit_name_build(const char *prefix, const char *instance, const char *suffix);
|