diff options
author | Lukas Slebodnik <lslebodn@redhat.com> | 2015-03-06 20:50:21 +0100 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2015-04-14 13:28:50 +0200 |
commit | 4ea6bc6dea87ac8cb37eb271ea86350e89695670 (patch) | |
tree | d968be71853e45a7d236895362aa1017100f5d05 | |
parent | 4a73eb4c8136f8e5fd47aa63e5b7c10afc10046e (diff) | |
download | sssd-4ea6bc6dea87ac8cb37eb271ea86350e89695670.tar.gz sssd-4ea6bc6dea87ac8cb37eb271ea86350e89695670.tar.xz sssd-4ea6bc6dea87ac8cb37eb271ea86350e89695670.zip |
sysdb-tests: test return value before output arguments
Output arguments needn't be initialized if function failed.
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
-rw-r--r-- | src/tests/sysdb-tests.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/tests/sysdb-tests.c b/src/tests/sysdb-tests.c index 7b150591b..a6c6b33de 100644 --- a/src/tests/sysdb-tests.c +++ b/src/tests/sysdb-tests.c @@ -5602,6 +5602,7 @@ START_TEST(test_autofs_get_duplicate_keys) ret = sysdb_search_entry(test_ctx, test_ctx->sysdb, dn, LDB_SCOPE_SUBTREE, filter, attrs, &count, &msgs); + fail_unless(ret == EOK, "sysdb_search_entry returned [%d]", ret); fail_if(count != expected, "Found %d entries with name %s, expected %d\n", count, autofskey, expected); talloc_free(test_ctx); |