[SCM] Samba Shared Repository - branch master updated - b7d2ff38f54250db52c495d26a2b2923e5dcc83e

[Stefan Metzmacher]

The branch, master has been updated
       via  b7d2ff38f54250db52c495d26a2b2923e5dcc83e (commit)
      from  4962a4459ba73535d39eaa0fb77fd6bbf3f9bd7a (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit b7d2ff38f54250db52c495d26a2b2923e5dcc83e
Author: Stefan Metzmacher <metze at samba.org>
Date:   Wed Sep 24 03:59:53 2008 +0200

    s4:libcli/smb_composite: we only check the signature when the server return OK
    
    We need to manually free the request, otherwise the timeout handler is
    triggered later.
    
    metze

-----------------------------------------------------------------------

Summary of changes:
 source4/libcli/smb_composite/sesssetup.c |    9 +++++++++
 1 files changed, 9 insertions(+), 0 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source4/libcli/smb_composite/sesssetup.c b/source4/libcli/smb_composite/sesssetup.c
index 92b49dc..645f536 100644
--- a/source4/libcli/smb_composite/sesssetup.c
+++ b/source4/libcli/smb_composite/sesssetup.c
@@ -97,6 +97,15 @@ static void request_handler(struct smbcli_request *req)
 	c->status = state->remote_status;
 	state->req = NULL;
 
+	/*
+	 * we only need to check the signature if the
+	 * NT_STATUS_OK is returned
+	 */
+	if (!NT_STATUS_IS_OK(state->remote_status)) {
+		talloc_free(check_req);
+		check_req = NULL;
+	}
+
 	switch (state->setup.old.level) {
 	case RAW_SESSSETUP_OLD:
 		state->io->out.vuid = state->setup.old.out.vuid;


-- 
Samba Shared Repository