diff options
author | Thomas Hindoe Paaboel Andersen <phomes@gmail.com> | 2014-09-28 18:46:15 +0200 |
---|---|---|
committer | Thomas Hindoe Paaboel Andersen <phomes@gmail.com> | 2014-09-28 18:48:33 +0200 |
commit | 5e90b6a978d15efedc5b5cc4a3d2d922a0ecd2a9 (patch) | |
tree | d9c70294bb07ed6dcc48a19c53b49f9d692de3e0 /src/bus-proxyd/test-bus-policy.c | |
parent | e931d3f4241231e4102eda06adaf7cbfd68c6a5d (diff) |
test-bus-policy: silence coverity
Check if strjoin worked and also use _cleanup_free_ since we are
here.
Found with Coverity. Fixes CID#1241962
Diffstat (limited to 'src/bus-proxyd/test-bus-policy.c')
-rw-r--r-- | src/bus-proxyd/test-bus-policy.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/bus-proxyd/test-bus-policy.c b/src/bus-proxyd/test-bus-policy.c index 900e4d2b06..7bcebef905 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; } |