summaryrefslogtreecommitdiff
path: root/source3/client/client.c
diff options
context:
space:
mode:
authorAndrew Tridgell <tridge@samba.org>2001-11-28 03:56:30 +0000
committerAndrew Tridgell <tridge@samba.org>2001-11-28 03:56:30 +0000
commiteec9e8a052407611df223fec982588e7a2bd7f49 (patch)
tree70742ae3dd80f86aca5b61afc079811d8aa2f826 /source3/client/client.c
parenta6b73b9767524fa6997bd31398d558a7cf1e4ac8 (diff)
downloadsamba-eec9e8a052407611df223fec982588e7a2bd7f49.tar.gz
samba-eec9e8a052407611df223fec982588e7a2bd7f49.tar.bz2
samba-eec9e8a052407611df223fec982588e7a2bd7f49.zip
fix a bunch of places where we can double-free a cli structure
(This used to be commit e2ba2383c9f679c076749a8f4fccefc3559e37ec)
Diffstat (limited to 'source3/client/client.c')
-rw-r--r--source3/client/client.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/source3/client/client.c b/source3/client/client.c
index 2b0867cf7b..c3d1fbc7ec 100644
--- a/source3/client/client.c
+++ b/source3/client/client.c
@@ -2124,7 +2124,6 @@ struct cli_state *do_connect(const char *server, const char *share)
d_printf("session request to %s failed (%s)\n",
called.name, cli_errstr(c));
cli_shutdown(c);
- SAFE_FREE(c);
if ((p=strchr_m(called.name, '.'))) {
*p = 0;
goto again;
@@ -2141,7 +2140,6 @@ struct cli_state *do_connect(const char *server, const char *share)
if (!cli_negprot(c)) {
d_printf("protocol negotiation failed\n");
cli_shutdown(c);
- SAFE_FREE(c);
return NULL;
}
@@ -2161,7 +2159,6 @@ struct cli_state *do_connect(const char *server, const char *share)
!cli_session_setup(c, "", "", 0, "", 0, workgroup)) {
d_printf("session setup failed: %s\n", cli_errstr(c));
cli_shutdown(c);
- SAFE_FREE(c);
return NULL;
}
d_printf("Anonymous login successful\n");
@@ -2186,7 +2183,6 @@ struct cli_state *do_connect(const char *server, const char *share)
password, strlen(password)+1)) {
d_printf("tree connect failed: %s\n", cli_errstr(c));
cli_shutdown(c);
- SAFE_FREE(c);
return NULL;
}