torture:smb2: fix crashes in smb2.durable-open.reopen1a test

If the test failed too early, we dereferenced tree2 which
was still NULL.

Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>

diff --git a/source4/torture/smb2/durable_open.c b/source4/torture/smb2/durable_open.c
index 11388ef3828e45397652bca60b664cf79bc01ed7..e8a82c0b3a4ebfda6a80728cb73668cb04ee0ed5 100644 (file)
--- a/source4/torture/smb2/durable_open.c
+++ b/source4/torture/smb2/durable_open.c
@@ -479,6 +479,8 @@ static bool test_durable_open_reopen1a(struct torture_context *tctx,
        status = smb2_create(tree, mem_ctx, &io2);
        CHECK_STATUS(status, NT_STATUS_USER_SESSION_DELETED);
 
+       TALLOC_FREE(tree);
+
        /*
         * but a durable reconnect on the new session succeeds:
         */
@@ -495,15 +497,19 @@ static bool test_durable_open_reopen1a(struct torture_context *tctx,
        h = &_h;
 
 done:
-       if (h != NULL) {
-               smb2_util_close(tree2, *h);
+       if (tree == NULL) {
+               tree = tree2;
        }
 
-       smb2_util_unlink(tree2, fname);
-
-       talloc_free(tree2);
+       if (tree != NULL) {
+               if (h != NULL) {
+                       smb2_util_close(tree, *h);
+                       h = NULL;
+               }
+               smb2_util_unlink(tree, fname);
 
-       talloc_free(tree);
+               talloc_free(tree);
+       }
 
        talloc_free(mem_ctx);