summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLukas Slebodnik <lslebodn@redhat.com>2015-03-06 20:50:21 +0100
committerJakub Hrozek <jhrozek@redhat.com>2015-04-14 13:28:50 +0200
commit4ea6bc6dea87ac8cb37eb271ea86350e89695670 (patch)
treed968be71853e45a7d236895362aa1017100f5d05
parent4a73eb4c8136f8e5fd47aa63e5b7c10afc10046e (diff)
downloadsssd-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.c1
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);