From e919cadcca063a4cb1a76a3c30b15f056a51e16d Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Fri, 18 May 2012 14:22:27 +1000 Subject: s4-torture: provide correct torture failures for failure to connect --- source4/torture/util_smb.c | 20 +++++++++----------- 1 file changed, 9 insertions(+), 11 deletions(-) (limited to 'source4/torture/util_smb.c') diff --git a/source4/torture/util_smb.c b/source4/torture/util_smb.c index 9efa33437cc..93993fd80f6 100644 --- a/source4/torture/util_smb.c +++ b/source4/torture/util_smb.c @@ -815,18 +815,17 @@ static bool wrap_simple_2smb_test(struct torture_context *torture_ctx, { bool (*fn) (struct torture_context *, struct smbcli_state *, struct smbcli_state *); - bool ret; + bool ret = true; - struct smbcli_state *cli1, *cli2; + struct smbcli_state *cli1 = NULL, *cli2 = NULL; - if (!torture_open_connection(&cli1, torture_ctx, 0) || - !torture_open_connection(&cli2, torture_ctx, 1)) - return false; + torture_assert_goto(torture_ctx, torture_open_connection(&cli1, torture_ctx, 0), ret, fail, "Failed to open connection"); + torture_assert_goto(torture_ctx, torture_open_connection(&cli2, torture_ctx, 1), ret, fail, "Failed to open connection"); fn = test->fn; ret = fn(torture_ctx, cli1, cli2); - +fail: talloc_free(cli1); talloc_free(cli2); @@ -866,17 +865,16 @@ static bool wrap_simple_1smb_test(struct torture_context *torture_ctx, struct torture_test *test) { bool (*fn) (struct torture_context *, struct smbcli_state *); - bool ret; + bool ret = true; - struct smbcli_state *cli1; + struct smbcli_state *cli1 = NULL; - if (!torture_open_connection(&cli1, torture_ctx, 0)) - return false; + torture_assert_goto(torture_ctx, torture_open_connection(&cli1, torture_ctx, 0), ret, fail, "Failed to open connection"); fn = test->fn; ret = fn(torture_ctx, cli1); - +fail: talloc_free(cli1); return ret; -- cgit v1.2.1