[PATCH 01/11] torture: Use torture_assert() macros in rpc.samr test

Jelmer Vernooij jelmer at samba.org
Fri Jun 20 04:57:34 MDT 2014


Reviewed-By: me

On Fri, Jun 20, 2014 at 10:28:43PM +1200, abartlet at samba.org wrote:
> From: Andrew Bartlett <abartlet at samba.org>
> 
> Change-Id: I844be928bb9309196e08898385ff0501efcc4e67
> Signed-off-by: Andrew Bartlett <abartlet at samba.org>
> Reviewed-by: Kamen Mazdrashki <kamenim at samba.org>
> ---
>  source4/torture/rpc/samr.c | 21 +++------------------
>  1 file changed, 3 insertions(+), 18 deletions(-)
> 
> diff --git a/source4/torture/rpc/samr.c b/source4/torture/rpc/samr.c
> index 0cdd242..7dfd183 100644
> --- a/source4/torture/rpc/samr.c
> +++ b/source4/torture/rpc/samr.c
> @@ -250,28 +250,13 @@ static bool test_SetUserInfo(struct dcerpc_binding_handle *b, struct torture_con
>  		}
>  
>  #define STRING_EQUAL(s1, s2, field) \
> -		if ((s1 && !s2) || (s2 && !s1) || strcmp(s1, s2)) { \
> -			torture_result(tctx, TORTURE_FAIL, "Failed to set %s to '%s' (%s)\n", \
> -			       #field, s2, __location__); \
> -			ret = false; \
> -			break; \
> -		}
> +	torture_assert_str_equal(tctx, s1, s2, "Failed to set " #field)
>  
>  #define MEM_EQUAL(s1, s2, length, field) \
> -		if ((s1 && !s2) || (s2 && !s1) || memcmp(s1, s2, length)) { \
> -			torture_result(tctx, TORTURE_FAIL, "Failed to set %s to '%s' (%s)\n", \
> -			       #field, (const char *)s2, __location__); \
> -			ret = false; \
> -			break; \
> -		}
> +	torture_assert_mem_equal(tctx, s1, s2, length, "Failed to set " #field)
>  
>  #define INT_EQUAL(i1, i2, field) \
> -		if (i1 != i2) { \
> -			torture_result(tctx, TORTURE_FAIL, "Failed to set %s to 0x%llx - got 0x%llx (%s)\n", \
> -			       #field, (unsigned long long)i2, (unsigned long long)i1, __location__); \
> -			ret = false; \
> -			break; \
> -		}
> +	torture_assert_int_equal(tctx, i1, i2, "Failed to set " #field)
>  
>  #define TEST_USERINFO_STRING(lvl1, field1, lvl2, field2, value, fpval) do { \
>  		torture_comment(tctx, "field test %d/%s vs %d/%s\n", lvl1, #field1, lvl2, #field2); \
> -- 
> 1.9.3
> 


More information about the samba-technical mailing list