From e54342b062e7736b46f56bcb0e064acc5a7b98ee Mon Sep 17 00:00:00 2001 From: Gerald Carter Date: Fri, 19 Mar 2004 15:52:16 +0000 Subject: [PATCH] wrap cli_tdis() in check to make sure we have a valid cnum --- source/libsmb/clientgen.c | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/source/libsmb/clientgen.c b/source/libsmb/clientgen.c index 28da73894fe..370249d0474 100644 --- a/source/libsmb/clientgen.c +++ b/source/libsmb/clientgen.c @@ -367,15 +367,16 @@ void cli_nt_netlogon_netsec_session_close(struct cli_state *cli) void cli_close_connection(struct cli_state *cli) { - /* - * tell our peer to free his resources. Wihtout this, when an - * application attempts to do a graceful shutdown and calls - * smbc_free_context() to clean up all connections, some connections - * can remain active on the peer end, until some (long) timeout period - * later. This tree disconnect forces the peer to clean up, since the - * connection will be going away. - */ - cli_tdis(cli); + /* + * tell our peer to free his resources. Wihtout this, when an + * application attempts to do a graceful shutdown and calls + * smbc_free_context() to clean up all connections, some connections + * can remain active on the peer end, until some (long) timeout period + * later. This tree disconnect forces the peer to clean up, since the + * connection will be going away. + */ + if ( cli->cnum != -1 ) + cli_tdis(cli); cli_nt_session_close(cli); cli_nt_netlogon_netsec_session_close(cli);