diff options
author | Michal Privoznik <mprivozn@redhat.com> | 2019-03-13 09:45:25 +0100 |
---|---|---|
committer | Michal Privoznik <mprivozn@redhat.com> | 2019-03-13 14:15:35 +0100 |
commit | 288e0ab1066def1348f5b2c6e63dce24a67e617d (patch) | |
tree | c00757d4d1190a0f0ca8edb76100b908866841ea /tests/virfilewrapper.c | |
parent | 8b71b0c727398469b95d025e3ec92b881969412f (diff) | |
download | libvirt-288e0ab1066def1348f5b2c6e63dce24a67e617d.tar.gz |
tests: Turn virFileWrapperAddPrefix to void
In theory, it's nice to have virFileWrapperAddPrefix() return a
value that indicates if the function succeeded or not. But in
practice, nobody checks for that and in fact blindly believes
that the function succeeded. Therefore, make the function return
nothing and just abort() if it would fail.
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Martin Kletzander <mkletzan@redhat.com>
Reviewed-by: Erik Skultety <eskultet@redhat.com>
Diffstat (limited to 'tests/virfilewrapper.c')
-rw-r--r-- | tests/virfilewrapper.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/tests/virfilewrapper.c b/tests/virfilewrapper.c index f7a7a931e1..497e91bd45 100644 --- a/tests/virfilewrapper.c +++ b/tests/virfilewrapper.c @@ -66,13 +66,15 @@ static void init_syms(void) } -int +void virFileWrapperAddPrefix(const char *prefix, const char *override) { /* Both parameters are mandatory */ - if (!prefix || !override) - return -1; + if (!prefix || !override) { + fprintf(stderr, "Attempt to add invalid path override\n"); + abort(); + } init_syms(); @@ -80,10 +82,9 @@ virFileWrapperAddPrefix(const char *prefix, VIR_APPEND_ELEMENT_QUIET(overrides, noverrides, override) < 0) { VIR_FREE(prefixes); VIR_FREE(overrides); - return -1; + fprintf(stderr, "Unable to add path override for '%s'\n", prefix); + abort(); } - - return 0; } |