[SCM] Samba Shared Repository - branch v4-0-test updated -
release-4-0-0alpha3-114-g9b8448c
Jelmer Vernooij
jelmer at samba.org
Sun Apr 13 20:49:45 GMT 2008
The branch, v4-0-test has been updated
via 9b8448c1e4567be26fa4890c647e0d4d46efbe20 (commit)
via e2c3fab9d1bf0482c15a115e7d373562ffe50b29 (commit)
via b6319b64d7ef9c58f771f338317d4d525d39a8f2 (commit)
via 10102d80d0f78777a69f6b3b1e5606d7d56b7254 (commit)
from 75c8dc6c6f3134bb78356630f24617aaeb869344 (commit)
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v4-0-test
- Log -----------------------------------------------------------------
commit 9b8448c1e4567be26fa4890c647e0d4d46efbe20
Merge: e2c3fab9d1bf0482c15a115e7d373562ffe50b29 75c8dc6c6f3134bb78356630f24617aaeb869344
Author: Jelmer Vernooij <jelmer at samba.org>
Date: Sun Apr 13 22:49:08 2008 +0200
Merge branch 'v4-0-test' of ssh://git.samba.org/data/git/samba into cprovision
commit e2c3fab9d1bf0482c15a115e7d373562ffe50b29
Author: Jelmer Vernooij <jelmer at samba.org>
Date: Sun Apr 13 22:00:36 2008 +0200
Use torture_assert_mem_equal() in a couple more places.
commit b6319b64d7ef9c58f771f338317d4d525d39a8f2
Author: Jelmer Vernooij <jelmer at samba.org>
Date: Sun Apr 13 21:33:59 2008 +0200
Use new torture_assert_mem_equal.
commit 10102d80d0f78777a69f6b3b1e5606d7d56b7254
Author: Jelmer Vernooij <jelmer at samba.org>
Date: Sun Apr 13 21:31:06 2008 +0200
Attempt to fix get_value() test on sparc machines.
-----------------------------------------------------------------------
Summary of changes:
source/lib/registry/tests/hive.c | 2 +-
source/lib/registry/tests/registry.c | 4 ++--
source/lib/socket/testsuite.c | 6 +++---
source/lib/util/tests/file.c | 4 ++--
source/torture/auth/ntlmssp.c | 4 ++--
source/torture/basic/base.c | 2 +-
source/torture/ndr/netlogon.c | 8 ++++----
source/torture/ui.h | 9 +++++++++
8 files changed, 24 insertions(+), 15 deletions(-)
Changeset truncated at 500 lines:
diff --git a/source/lib/registry/tests/hive.c b/source/lib/registry/tests/hive.c
index 70b0241..a16736c 100644
--- a/source/lib/registry/tests/hive.c
+++ b/source/lib/registry/tests/hive.c
@@ -220,7 +220,7 @@ static bool test_get_value(struct torture_context *tctx, const void *test_data)
torture_assert_int_equal(tctx, value.length, 4, "value length");
torture_assert_int_equal(tctx, type, REG_DWORD, "value type");
- torture_assert_int_equal(tctx, data, IVAL(value.data, 0),
+ torture_assert_mem_equal(tctx, &data, value.data, sizeof(uint32_t),
"value data");
return true;
diff --git a/source/lib/registry/tests/registry.c b/source/lib/registry/tests/registry.c
index e5e34c1..b19a6ab 100644
--- a/source/lib/registry/tests/registry.c
+++ b/source/lib/registry/tests/registry.c
@@ -428,7 +428,7 @@ static bool test_get_value(struct torture_context *tctx, void *_data)
torture_assert_werr_ok(tctx, error, "getting value");
torture_assert_int_equal(tctx, 4, data.length, "value length ok");
- torture_assert(tctx, memcmp(data.data, &value, 4) == 0,
+ torture_assert_mem_equal(tctx, data.data, &value, 4,
"value content ok");
torture_assert_int_equal(tctx, REG_DWORD, type, "value type");
@@ -496,7 +496,7 @@ static bool test_list_values(struct torture_context *tctx, void *_data)
torture_assert_str_equal(tctx, name, "bar", "value name");
torture_assert_int_equal(tctx, 4, data.length, "value length");
- torture_assert(tctx, memcmp(data.data, &value, 4) == 0,
+ torture_assert_mem_equal(tctx, data.data, &value, 4,
"value content");
torture_assert_int_equal(tctx, REG_DWORD, type, "value type");
diff --git a/source/lib/socket/testsuite.c b/source/lib/socket/testsuite.c
index 6d4d81a..813412c 100644
--- a/source/lib/socket/testsuite.c
+++ b/source/lib/socket/testsuite.c
@@ -86,7 +86,7 @@ static bool test_udp(struct torture_context *tctx)
torture_assert_int_equal(tctx, nread, size, "Unexpected recvfrom size");
- torture_assert(tctx, memcmp(blob2.data, blob.data, size) == 0,
+ torture_assert_mem_equal(tctx, blob2.data, blob.data, size,
"Bad data in recvfrom");
generate_random_buffer(blob.data, blob.length);
@@ -104,7 +104,7 @@ static bool test_udp(struct torture_context *tctx)
torture_assert_int_equal(tctx, from_addr->port, srv_addr->port,
"Unexpected recvfrom port");
- torture_assert(tctx, memcmp(blob2.data, blob.data, size) == 0,
+ torture_assert_mem_equal(tctx, blob2.data, blob.data, size,
"Bad data in recvfrom");
talloc_free(sock1);
@@ -181,7 +181,7 @@ static bool test_tcp(struct torture_context *tctx)
torture_assert_int_equal(tctx, nread, size, "Unexpected recvfrom size");
- torture_assert(tctx, memcmp(blob2.data, blob.data, size) == 0,
+ torture_assert_mem_equal(tctx, blob2.data, blob.data, size,
"Bad data in recv");
return true;
}
diff --git a/source/lib/util/tests/file.c b/source/lib/util/tests/file.c
index fe87293..3377e83 100644
--- a/source/lib/util/tests/file.c
+++ b/source/lib/util/tests/file.c
@@ -45,9 +45,9 @@ static bool test_file_load_save(struct torture_context *tctx)
data = file_load(TEST_FILENAME, &len, mem_ctx);
torture_assert(tctx, data, "loading file");
- torture_assert(tctx, len == strlen(TEST_DATA), "Length");
+ torture_assert_int_equal(tctx, len, strlen(TEST_DATA), "Length");
- torture_assert(tctx, memcmp(data, TEST_DATA, len) == 0, "Contents");
+ torture_assert_mem_equal(tctx, data, TEST_DATA, len, "Contents");
unlink(TEST_FILENAME);
return true;
diff --git a/source/torture/auth/ntlmssp.c b/source/torture/auth/ntlmssp.c
index 9366de9..917a24a 100644
--- a/source/torture/auth/ntlmssp.c
+++ b/source/torture/auth/ntlmssp.c
@@ -69,7 +69,7 @@ static bool torture_ntlmssp_self_check(struct torture_context *tctx)
torture_assert_int_equal(tctx, sig.length, expected_sig.length, "Wrong sig length");
- torture_assert(tctx, 0 == memcmp(sig.data, expected_sig.data, sig.length),
+ torture_assert_mem_equal(tctx, sig.data, expected_sig.data, sig.length,
"data mismatch");
torture_assert_ntstatus_equal(tctx,
@@ -123,7 +123,7 @@ static bool torture_ntlmssp_self_check(struct torture_context *tctx)
torture_assert_int_equal(tctx, sig.length, expected_sig.length, "Wrong sig length");
- torture_assert(tctx, 0 == memcmp(sig.data+8, expected_sig.data+8, sig.length-8),
+ torture_assert_mem_equal(tctx, sig.data+8, expected_sig.data+8, sig.length-8,
"data mismatch");
torture_assert_ntstatus_equal(tctx,
diff --git a/source/torture/basic/base.c b/source/torture/basic/base.c
index ac71d14..3a3a7c2 100644
--- a/source/torture/basic/base.c
+++ b/source/torture/basic/base.c
@@ -590,7 +590,7 @@ static bool rw_torture2(struct torture_context *tctx,
break;
}
- torture_assert(tctx, memcmp(buf_rd, buf, buf_size) == 0,
+ torture_assert_mem_equal(tctx, buf_rd, buf, buf_size,
"read/write compare failed\n");
}
diff --git a/source/torture/ndr/netlogon.c b/source/torture/ndr/netlogon.c
index 2f01a9d..f498335 100644
--- a/source/torture/ndr/netlogon.c
+++ b/source/torture/ndr/netlogon.c
@@ -46,7 +46,7 @@ static bool netrserverauthenticate3_in_check(struct torture_context *tctx,
torture_assert_int_equal(tctx, r->in.secure_channel_type, 2, "secure channel type");
torture_assert_str_equal(tctx, r->in.computer_name, "NATIVE-2K", "computer name");
torture_assert_int_equal(tctx, *r->in.negotiate_flags, 0x6007ffff, "negotiate flags");
- torture_assert(tctx, memcmp(cred_expected, r->in.credentials->data, 8) == 0, "credentials");
+ torture_assert_mem_equal(tctx, cred_expected, r->in.credentials->data, 8, "credentials");
return true;
}
@@ -59,7 +59,7 @@ static bool netrserverauthenticate3_out_check(struct torture_context *tctx,
struct netr_ServerAuthenticate3 *r)
{
uint8_t cred_expected[8] = { 0x22, 0x0c, 0x86, 0x8a, 0xe9, 0x92, 0x93, 0xc9 };
- torture_assert(tctx, memcmp(cred_expected, r->out.credentials->data, 8) == 0, "credentials");
+ torture_assert_mem_equal(tctx, cred_expected, r->out.credentials->data, 8, "credentials");
torture_assert_int_equal(tctx, *r->out.negotiate_flags, 0x6007ffff, "negotiate flags");
torture_assert_int_equal(tctx, *r->out.rid, 0x454, "rid");
torture_assert_ntstatus_ok(tctx, r->out.result, "return code");
@@ -85,7 +85,7 @@ static bool netrserverreqchallenge_in_check(struct torture_context *tctx,
uint8_t cred_expected[8] = { 0xa3, 0x2c, 0xa2, 0x95, 0x40, 0xcc, 0xb7, 0xbb };
torture_assert_str_equal(tctx, r->in.server_name, "\\\\NATIVE-DC.NATIVE.BASE", "server name");
torture_assert_str_equal(tctx, r->in.computer_name, "NATIVE-2K", "account name");
- torture_assert(tctx, memcmp(cred_expected, r->in.credentials->data, 8) == 0, "credentials");
+ torture_assert_mem_equal(tctx, cred_expected, r->in.credentials->data, 8, "credentials");
return true;
}
@@ -98,7 +98,7 @@ static bool netrserverreqchallenge_out_check(struct torture_context *tctx,
struct netr_ServerReqChallenge *r)
{
uint8_t cred_expected[8] = { 0x22, 0xfc, 0xc1, 0x17, 0xc0, 0xae, 0x27, 0x8e };
- torture_assert(tctx, memcmp(cred_expected, r->out.credentials->data, 8) == 0, "credentials");
+ torture_assert_mem_equal(tctx, cred_expected, r->out.credentials->data, 8, "credentials");
torture_assert_ntstatus_ok(tctx, r->out.result, "return code");
return true;
diff --git a/source/torture/ui.h b/source/torture/ui.h
index 6645d9b..ec4a658 100644
--- a/source/torture/ui.h
+++ b/source/torture/ui.h
@@ -253,6 +253,15 @@ void torture_result(struct torture_context *test,
} \
} while(0)
+#define torture_assert_mem_equal(torture_ctx,got,expected,len,cmt)\
+ do { const void *__got = (got), *__expected = (expected); \
+ if (memcmp(__got, __expected, len) != 0) { \
+ torture_result(torture_ctx, TORTURE_FAIL, \
+ __location__": "#got" of len %d did not match"#expected": %s", len, cmt); \
+ return false; \
+ } \
+ } while(0)
+
#define torture_assert_file_contains_text(torture_ctx,filename,expected,cmt)\
do { \
char *__got; \
--
Samba Shared Repository
More information about the samba-cvs
mailing list