From 5e90b6a978d15efedc5b5cc4a3d2d922a0ecd2a9 Mon Sep 17 00:00:00 2001 From: Thomas Hindoe Paaboel Andersen Date: Sun, 28 Sep 2014 18:46:15 +0200 Subject: [PATCH 1/1] test-bus-policy: silence coverity Check if strjoin worked and also use _cleanup_free_ since we are here. Found with Coverity. Fixes CID#1241962 --- src/bus-proxyd/test-bus-policy.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/bus-proxyd/test-bus-policy.c b/src/bus-proxyd/test-bus-policy.c index 900e4d2b0..7bcebef90 100644 --- a/src/bus-proxyd/test-bus-policy.c +++ b/src/bus-proxyd/test-bus-policy.c @@ -46,18 +46,17 @@ static int test_policy_load(Policy *p, const char *name) { - char *path; + _cleanup_free_ char *path = NULL; int r = 0; path = strjoin(TEST_DIR, "/bus-policy/", name, NULL); + assert_se(path); if (access(path, R_OK) == 0) policy_load(p, STRV_MAKE(path)); else r = -ENOENT; - free(path); - return r; } -- 2.30.2