s3:client: Fix a use-after-free issue in smbclient

Detected by

    make test TESTS="samba3.blackbox.chdir-cache"

with an optimized build or with AddressSanitizer.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=15268

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
(cherry picked from commit 9c707b4be27e2a6f79886d3ec8b5066c922b99bd)

Autobuild-User(v4-16-test): Jule Anger <janger@samba.org>
Autobuild-Date(v4-16-test): Tue Jan  3 19:19:57 UTC 2023 on sn-devel-184

diff --git a/source3/client/client.c b/source3/client/client.c
index 69b7c9022c2e89497704f02aac520a5c4d2cc96e..607be92bba3b8c986dcd46fb7621a08b738cb603 100644 (file)
--- a/source3/client/client.c
+++ b/source3/client/client.c
@@ -5126,10 +5126,11 @@ static int cmd_tcon(void)
                return -1;
        }
 
-       talloc_free(sharename);
-
        d_printf("tcon to %s successful, tid: %u\n", sharename,
                 cli_state_get_tid(cli));
+
+       talloc_free(sharename);
+
        return 0;
 }