]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
test: fix condition test if there are no controllers
authorLennart Poettering <lennart@poettering.net>
Wed, 20 Dec 2017 17:13:23 +0000 (18:13 +0100)
committerLennart Poettering <lennart@poettering.net>
Fri, 5 Jan 2018 12:58:32 +0000 (13:58 +0100)
As an optimization cg_mask_to_string() returns NULL if there are no
controllers available. We need to handle that.

src/test/test-condition.c

index 8323a66ad39b0483e614b36bccfa65e399ee073c..ad64a2bb36ec0471b2e8e0af9d66249648ac8415 100644 (file)
@@ -29,8 +29,8 @@
 #include "apparmor-util.h"
 #include "architecture.h"
 #include "audit-util.h"
-#include "condition.h"
 #include "cgroup-util.h"
+#include "condition.h"
 #include "hostname-util.h"
 #include "id128-util.h"
 #include "ima-util.h"
@@ -187,12 +187,12 @@ static int test_condition_test_control_group_controller(void) {
         /* Multiple valid controllers at the same time */
         assert_se(cg_mask_to_string(system_mask, &controller_name) >= 0);
 
-        condition = condition_new(CONDITION_CONTROL_GROUP_CONTROLLER, controller_name, false, false);
+        condition = condition_new(CONDITION_CONTROL_GROUP_CONTROLLER, strempty(controller_name), false, false);
         assert_se(condition);
         assert_se(condition_test(condition));
         condition_free(condition);
 
-        condition = condition_new(CONDITION_CONTROL_GROUP_CONTROLLER, controller_name, false, true);
+        condition = condition_new(CONDITION_CONTROL_GROUP_CONTROLLER, strempty(controller_name), false, true);
         assert_se(condition);
         assert_se(!condition_test(condition));
         condition_free(condition);