summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGary Lockyer <gary@catalyst.net.nz>2018-02-23 15:03:20 +1300
committerStefan Metzmacher <metze@samba.org>2018-03-01 00:42:25 +0100
commit93cfa4634745d60797de9346ad56cc9d8c3f6122 (patch)
tree0ed75fddc8181f5e15cd44f13185521f2a145c71
parent105a5b09f2160fb2092e4eb4a2960f1a9bfc93c4 (diff)
downloadsamba-93cfa4634745d60797de9346ad56cc9d8c3f6122.tar.gz
ldb tests: fix null test on incorrect variable
Fix up tests that were performing a null check on the wrong variable after a call to ldb_msg_new Signed-off-by: Gary Lockyer <gary@catalyst.net.nz> Reviewed-by: Andrew Bartlett <abartlet@samba.org> Autobuild-User(master): Andrew Bartlett <abartlet@samba.org> Autobuild-Date(master): Sat Feb 24 15:50:35 CET 2018 on sn-devel-144 (cherry picked from commit 1ed693423d7a30c3810ddc1b6f052d376c8cd4e7)
-rw-r--r--lib/ldb/tests/ldb_mod_op_test.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/ldb/tests/ldb_mod_op_test.c b/lib/ldb/tests/ldb_mod_op_test.c
index cf2288c7bce..0190936231b 100644
--- a/lib/ldb/tests/ldb_mod_op_test.c
+++ b/lib/ldb/tests/ldb_mod_op_test.c
@@ -3226,7 +3226,7 @@ static void test_ldb_add_duplicate_value_to_unique_index(void **state)
assert_int_equal(ret, LDB_SUCCESS);
msg02 = ldb_msg_new(tmp_ctx);
- assert_non_null(msg01);
+ assert_non_null(msg02);
msg02->dn = ldb_dn_new_fmt(msg02, test_ctx->ldb, "dc=test02");
assert_non_null(msg02->dn);
@@ -3267,7 +3267,7 @@ static void test_ldb_add_to_index_duplicates_allowed(void **state)
assert_int_equal(ret, LDB_SUCCESS);
msg02 = ldb_msg_new(tmp_ctx);
- assert_non_null(msg01);
+ assert_non_null(msg02);
msg02->dn = ldb_dn_new_fmt(msg02, test_ctx->ldb, "dc=test02");
assert_non_null(msg02->dn);