test: Remove an unnecessary NULL check in sd-activation
authorPhilip Withnall <withnall@endlessm.com>
Tue, 28 Feb 2017 08:56:25 +0000 (08:56 +0000)
committerSimon McVittie <smcv@debian.org>
Mon, 20 Mar 2017 14:54:40 +0000 (14:54 +0000)
commitd42905bc04ee781771552562cfff370a0108541e
tree6d00f6c8b919206bee000e2fefd27aa86914656f
parentf6cee5f0467fac38ce0bc21218b0cdc613dd5306
test: Remove an unnecessary NULL check in sd-activation

test_transient_services() was checking whether (config == NULL), then
shortly afterwards dereferencing config->bus_name (which is in fact
dereferenced on all branches of the function). Remove the check and add
some assertions to ensure a Config struct is always passed into the test
(which it currently is).

Coverity ID: 141877

Signed-off-by: Philip Withnall <withnall@endlessm.com>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=99999
Reviewed-by: Simon McVittie <smcv@collabora.com>
test/sd-activation.c