summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLennart Poettering <lennart@poettering.net>2014-11-06 02:02:13 +0100
committerLennart Poettering <lennart@poettering.net>2014-11-06 14:21:10 +0100
commita4705396adbf1a8a3e52ed133d9b1f12cb13e77e (patch)
tree46818d00ad1b0eff9cb83658be87ac6c3b39233a
parentd1bddcec98551ea748f39a742a4cbcf9ea9254ef (diff)
condition: internalize condition test functions
Also, implement the negation check inside of condition_test() instead of individually in each test function.
-rw-r--r--Makefile.am15
-rw-r--r--src/libsystemd-network/network-internal.c8
-rw-r--r--src/shared/condition-util.c91
-rw-r--r--src/shared/condition-util.h6
4 files changed, 60 insertions, 60 deletions
diff --git a/Makefile.am b/Makefile.am
index 452f07ca75..c5667c3a63 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -3488,7 +3488,8 @@ systemd_udevd_SOURCES = \
src/udev/udevd.c
systemd_udevd_LDADD = \
- libudev-core.la
+ libudev-core.la \
+ libsystemd-capability.la
udevadm_SOURCES = \
src/udev/udevadm.c \
@@ -3502,7 +3503,8 @@ udevadm_SOURCES = \
src/udev/udevadm-test-builtin.c
udevadm_LDADD = \
- libudev-core.la
+ libudev-core.la \
+ libsystemd-capability.la
# Update hwdb on installation. Do not bother if installing
# in DESTDIR, since this is likely for packaging purposes.
@@ -3536,7 +3538,8 @@ test_udev_SOURCES = \
src/test/test-udev.c
test_udev_LDADD = \
- libudev-core.la \
+ libudev-core.la \
+ libsystemd-capability.la \
$(BLKID_LIBS) \
$(KMOD_LIBS) \
$(SELINUX_LIBS)
@@ -5210,11 +5213,9 @@ networkctl_LDADD = \
test_network_SOURCES = \
src/network/test-network.c
-test_network_CFLAGS = \
- $(AM_CFLAGS)
-
test_network_LDADD = \
- libsystemd-networkd-core.la
+ libsystemd-networkd-core.la \
+ libsystemd-capability.la
test_network_tables_SOURCES = \
src/network/test-network-tables.c \
diff --git a/src/libsystemd-network/network-internal.c b/src/libsystemd-network/network-internal.c
index 372f3ed371..6f16050cdc 100644
--- a/src/libsystemd-network/network-internal.c
+++ b/src/libsystemd-network/network-internal.c
@@ -98,16 +98,16 @@ bool net_match_config(const struct ether_addr *match_mac,
const char *dev_type,
const char *dev_name) {
- if (match_host && !condition_test_host(match_host))
+ if (match_host && !condition_test(match_host))
return 0;
- if (match_virt && !condition_test_virtualization(match_virt))
+ if (match_virt && !condition_test(match_virt))
return 0;
- if (match_kernel && !condition_test_kernel_command_line(match_kernel))
+ if (match_kernel && !condition_test(match_kernel))
return 0;
- if (match_arch && !condition_test_architecture(match_arch))
+ if (match_arch && !condition_test(match_arch))
return 0;
if (match_mac && (!dev_mac || memcmp(match_mac, dev_mac, ETH_ALEN)))
diff --git a/src/shared/condition-util.c b/src/shared/condition-util.c
index ab889e3222..7569b40d6d 100644
--- a/src/shared/condition-util.c
+++ b/src/shared/condition-util.c
@@ -78,7 +78,7 @@ void condition_free_list(Condition *first) {
condition_free(c);
}
-int condition_test_kernel_command_line(Condition *c) {
+static int condition_test_kernel_command_line(Condition *c) {
_cleanup_free_ char *line = NULL;
const char *p;
bool equal;
@@ -92,7 +92,7 @@ int condition_test_kernel_command_line(Condition *c) {
if (r < 0)
return r;
if (r == 0)
- return c->negate;
+ return false;
equal = !!strchr(c->parameter, '=');
p = line;
@@ -105,7 +105,7 @@ int condition_test_kernel_command_line(Condition *c) {
if (r < 0)
return r;
if (r == 0)
- return c->negate;
+ break;
if (equal)
found = streq(word, c->parameter);
@@ -117,13 +117,13 @@ int condition_test_kernel_command_line(Condition *c) {
}
if (found)
- return !c->negate;
+ return true;
}
- return c->negate;
+ return false;
}
-int condition_test_virtualization(Condition *c) {
+static int condition_test_virtualization(Condition *c) {
int b, v;
const char *id;
@@ -139,23 +139,23 @@ int condition_test_virtualization(Condition *c) {
b = parse_boolean(c->parameter);
if (v > 0 && b > 0)
- return !c->negate;
+ return true;
if (v == 0 && b == 0)
- return !c->negate;
+ return true;
/* Then, compare categorization */
if (v == VIRTUALIZATION_VM && streq(c->parameter, "vm"))
- return !c->negate;
+ return true;
if (v == VIRTUALIZATION_CONTAINER && streq(c->parameter, "container"))
- return !c->negate;
+ return true;
/* Finally compare id */
- return (v > 0 && streq(c->parameter, id)) == !c->negate;
+ return v > 0 && streq(c->parameter, id);
}
-int condition_test_architecture(Condition *c) {
+static int condition_test_architecture(Condition *c) {
int a, b;
assert(c);
@@ -173,10 +173,10 @@ int condition_test_architecture(Condition *c) {
if (b < 0)
return b;
- return (a == b) == !c->negate;
+ return a == b;
}
-int condition_test_host(Condition *c) {
+static int condition_test_host(Condition *c) {
_cleanup_free_ char *h = NULL;
sd_id128_t x, y;
int r;
@@ -191,17 +191,17 @@ int condition_test_host(Condition *c) {
if (r < 0)
return r;
- return sd_id128_equal(x, y) == !c->negate;
+ return sd_id128_equal(x, y);
}
h = gethostname_malloc();
if (!h)
return -ENOMEM;
- return (fnmatch(c->parameter, h, FNM_CASEFOLD) == 0) == !c->negate;
+ return fnmatch(c->parameter, h, FNM_CASEFOLD) == 0;
}
-int condition_test_ac_power(Condition *c) {
+static int condition_test_ac_power(Condition *c) {
int r;
assert(c);
@@ -212,7 +212,7 @@ int condition_test_ac_power(Condition *c) {
if (r < 0)
return r;
- return ((on_ac_power() != 0) == !!r) == !c->negate;
+ return (on_ac_power() != 0) == !!r;
}
static int condition_test_security(Condition *c) {
@@ -221,17 +221,17 @@ static int condition_test_security(Condition *c) {
assert(c->type == CONDITION_SECURITY);
if (streq(c->parameter, "selinux"))
- return mac_selinux_use() == !c->negate;
+ return mac_selinux_use();
if (streq(c->parameter, "smack"))
- return mac_smack_use() == !c->negate;
+ return mac_smack_use();
if (streq(c->parameter, "apparmor"))
- return mac_apparmor_use() == !c->negate;
+ return mac_apparmor_use();
if (streq(c->parameter, "audit"))
- return use_audit() == !c->negate;
+ return use_audit();
if (streq(c->parameter, "ima"))
- return use_ima() == !c->negate;
+ return use_ima();
- return c->negate;
+ return false;
}
static int condition_test_capability(Condition *c) {
@@ -265,7 +265,7 @@ static int condition_test_capability(Condition *c) {
}
}
- return !!(capabilities & (1ULL << value)) == !c->negate;
+ return !!(capabilities & (1ULL << value));
}
static int condition_test_needs_update(Condition *c) {
@@ -278,24 +278,24 @@ static int condition_test_needs_update(Condition *c) {
/* If the file system is read-only we shouldn't suggest an update */
if (path_is_read_only_fs(c->parameter) > 0)
- return c->negate;
+ return false;
/* Any other failure means we should allow the condition to be true,
* so that we rather invoke too many update tools then too
* few. */
if (!path_is_absolute(c->parameter))
- return !c->negate;
+ return true;
p = strappenda(c->parameter, "/.updated");
if (lstat(p, &other) < 0)
- return !c->negate;
+ return true;
if (lstat("/usr/", &usr) < 0)
- return !c->negate;
+ return true;
- return (usr.st_mtim.tv_sec > other.st_mtim.tv_sec ||
- (usr.st_mtim.tv_sec == other.st_mtim.tv_sec && usr.st_mtim.tv_nsec > other.st_mtim.tv_nsec)) == !c->negate;
+ return usr.st_mtim.tv_sec > other.st_mtim.tv_sec ||
+ (usr.st_mtim.tv_sec == other.st_mtim.tv_sec && usr.st_mtim.tv_nsec > other.st_mtim.tv_nsec);
}
static int condition_test_first_boot(Condition *c) {
@@ -309,7 +309,7 @@ static int condition_test_first_boot(Condition *c) {
if (r < 0)
return r;
- return ((access("/run/systemd/first-boot", F_OK) >= 0) == !!r) == !c->negate;
+ return (access("/run/systemd/first-boot", F_OK) >= 0) == !!r;
}
static int condition_test_path_exists(Condition *c) {
@@ -317,7 +317,7 @@ static int condition_test_path_exists(Condition *c) {
assert(c->parameter);
assert(c->type == CONDITION_PATH_EXISTS);
- return (access(c->parameter, F_OK) >= 0) == !c->negate;
+ return access(c->parameter, F_OK) >= 0;
}
static int condition_test_path_exists_glob(Condition *c) {
@@ -325,7 +325,7 @@ static int condition_test_path_exists_glob(Condition *c) {
assert(c->parameter);
assert(c->type == CONDITION_PATH_EXISTS_GLOB);
- return (glob_exists(c->parameter) > 0) == !c->negate;
+ return glob_exists(c->parameter) > 0;
}
static int condition_test_path_is_directory(Condition *c) {
@@ -333,7 +333,7 @@ static int condition_test_path_is_directory(Condition *c) {
assert(c->parameter);
assert(c->type == CONDITION_PATH_IS_DIRECTORY);
- return (is_dir(c->parameter, true) > 0) == !c->negate;
+ return is_dir(c->parameter, true) > 0;
}
static int condition_test_path_is_symbolic_link(Condition *c) {
@@ -341,7 +341,7 @@ static int condition_test_path_is_symbolic_link(Condition *c) {
assert(c->parameter);
assert(c->type == CONDITION_PATH_IS_SYMBOLIC_LINK);
- return (is_symlink(c->parameter) > 0) == !c->negate;
+ return is_symlink(c->parameter) > 0;
}
static int condition_test_path_is_mount_point(Condition *c) {
@@ -349,7 +349,7 @@ static int condition_test_path_is_mount_point(Condition *c) {
assert(c->parameter);
assert(c->type == CONDITION_PATH_IS_MOUNT_POINT);
- return (path_is_mount_point(c->parameter, true) > 0) == !c->negate;
+ return path_is_mount_point(c->parameter, true) > 0;
}
static int condition_test_path_is_read_write(Condition *c) {
@@ -357,7 +357,7 @@ static int condition_test_path_is_read_write(Condition *c) {
assert(c->parameter);
assert(c->type == CONDITION_PATH_IS_READ_WRITE);
- return (path_is_read_only_fs(c->parameter) > 0) == c->negate;
+ return path_is_read_only_fs(c->parameter) <= 0;
}
static int condition_test_directory_not_empty(Condition *c) {
@@ -368,7 +368,7 @@ static int condition_test_directory_not_empty(Condition *c) {
assert(c->type == CONDITION_DIRECTORY_NOT_EMPTY);
r = dir_is_empty(c->parameter);
- return !(r == -ENOENT || r > 0) == !c->negate;
+ return r <= 0 && r != -ENOENT;
}
static int condition_test_file_not_empty(Condition *c) {
@@ -380,7 +380,7 @@ static int condition_test_file_not_empty(Condition *c) {
return (stat(c->parameter, &st) >= 0 &&
S_ISREG(st.st_mode) &&
- st.st_size > 0) == !c->negate;
+ st.st_size > 0);
}
static int condition_test_file_is_executable(Condition *c) {
@@ -392,7 +392,7 @@ static int condition_test_file_is_executable(Condition *c) {
return (stat(c->parameter, &st) >= 0 &&
S_ISREG(st.st_mode) &&
- (st.st_mode & 0111)) == !c->negate;
+ (st.st_mode & 0111));
}
static int condition_test_null(Condition *c) {
@@ -402,7 +402,7 @@ static int condition_test_null(Condition *c) {
/* Note that during parsing we already evaluate the string and
* store it in c->negate */
- return !c->negate;
+ return true;
}
int condition_test(Condition *c) {
@@ -428,12 +428,17 @@ int condition_test(Condition *c) {
[CONDITION_FIRST_BOOT] = condition_test_first_boot,
[CONDITION_NULL] = condition_test_null,
};
+ int r;
assert(c);
assert(c->type >= 0);
assert(c->type < _CONDITION_TYPE_MAX);
- return condition_tests[c->type](c);
+ r = condition_tests[c->type](c);
+ if (r < 0)
+ return r;
+
+ return (r > 0) == !c->negate;
}
void condition_dump(Condition *c, FILE *f, const char *prefix) {
diff --git a/src/shared/condition-util.h b/src/shared/condition-util.h
index deeb6b4257..44f672b2e6 100644
--- a/src/shared/condition-util.h
+++ b/src/shared/condition-util.h
@@ -77,12 +77,6 @@ Condition* condition_new(ConditionType type, const char *parameter, bool trigger
void condition_free(Condition *c);
void condition_free_list(Condition *c);
-int condition_test_kernel_command_line(Condition *c);
-int condition_test_virtualization(Condition *c);
-int condition_test_architecture(Condition *c);
-int condition_test_host(Condition *c);
-int condition_test_ac_power(Condition *c);
-
int condition_test(Condition *c);
void condition_dump(Condition *c, FILE *f, const char *prefix);