Fixed a memory leak in cli_set_mntpoint().

Fixes bug #10759 - Memory leak in libsmbclient in cli_set_mntpoint function

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

Signed-off-by: Har Gagan Sahai <SHarGagan@novell.com>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed Aug 13 04:36:50 CEST 2014 on sn-devel-104

diff --git a/source3/libsmb/clidfs.c b/source3/libsmb/clidfs.c
index 95f8817a94b70765027f1340f5b0ee45f66aba85..b2e2e9ec2b085d85318a29815b39e0e6940c6fd4 100644 (file)
--- a/source3/libsmb/clidfs.c
+++ b/source3/libsmb/clidfs.c
@@ -258,13 +258,15 @@ static NTSTATUS do_connect(TALLOC_CTX *ctx,
 
 static void cli_set_mntpoint(struct cli_state *cli, const char *mnt)
 {
-       char *name = clean_name(NULL, mnt);
+       TALLOC_CTX *frame = talloc_stackframe();
+       char *name = clean_name(frame, mnt);
        if (!name) {
+               TALLOC_FREE(frame);
                return;
        }
        TALLOC_FREE(cli->dfs_mountpoint);
        cli->dfs_mountpoint = talloc_strdup(cli, name);
-       TALLOC_FREE(name);
+       TALLOC_FREE(frame);
 }
 
 /********************************************************************