[SCM] Samba Shared Repository - branch master updated

Volker Lendecke vlendec at samba.org
Wed Aug 31 09:59:02 MDT 2011


The branch, master has been updated
       via  7810746 s3: Fix bug 8334, do not fork the echo handler for smb2
      from  da23473 s3-rpc_server: Disable epmapper by default.

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


- Log -----------------------------------------------------------------
commit 781074664deecec5c0067fccf1baee602fd18a83
Author: Volker Lendecke <vl at samba.org>
Date:   Wed Aug 31 15:06:35 2011 +0200

    s3: Fix bug 8334, do not fork the echo handler for smb2
    
    If a smb1 negprot negotiated smb2 we forked the echo responder. This will
    eventually lead to a panic from
    
    [2011/08/30 10:33:29.212578,  0, pid=3846917] smbd/smb2_server.c:243(smbd_smb2_request_create)
      Invalid SMB packet: first request: 0x0009
    
    because from the echo responder we always read using the normal smb1 protocol
    handling routine. If that is a bit down the smb2 stream, we get a non-negprot
    packet and panic.
    
    BTW, the echo responder is not required for smb2 anyway, Microsoft confirmed
    that it probes the server liveness using TCP keepalives and not smb2 echo
    requests.
    
    Autobuild-User: Volker Lendecke <vlendec at samba.org>
    Autobuild-Date: Wed Aug 31 17:58:48 CEST 2011 on sn-devel-104

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

Summary of changes:
 source3/smbd/negprot.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/smbd/negprot.c b/source3/smbd/negprot.c
index a587442..236b4d6 100644
--- a/source3/smbd/negprot.c
+++ b/source3/smbd/negprot.c
@@ -743,7 +743,8 @@ void reply_negprot(struct smb_request *req)
 
 	TALLOC_FREE(cliprotos);
 
-	if (lp_async_smb_echo_handler() && !fork_echo_handler(sconn)) {
+	if (lp_async_smb_echo_handler() && (get_Protocol() < PROTOCOL_SMB2) &&
+	    !fork_echo_handler(sconn)) {
 		exit_server("Failed to fork echo handler");
 	}
 


-- 
Samba Shared Repository


More information about the samba-cvs mailing list