CVS update: samba/source/libsmb

crh at samba.org crh at samba.org
Mon Jul 1 13:42:28 EST 2002


Date:	Sun Jun 30 20:42:28 2002
Author:	crh

Update of /data/cvs/samba/source/libsmb
In directory va:/tmp/cvs-serv22222/source/libsmb

Modified Files:
      Tag: SAMBA_3_0
	cliconnect.c 
Log Message:
The 17-bit length field in the header contains the number of
bytes which follow the header, not the full packet size.

    [Yes, the length field is either 17-bits, or (per the RFCs) it is a
    16-bit length field preceeded by an 8-bit flags field of which only
    the low-order bit may be used.  If that bit is set, then add 65536 to
    the 16-bit length field.  (In other words, it's a 17-bit unsigned
    length field.)
    ...unless, of course, the transport is native TCP [port 445] in which
    case the length field *might* be 24-bits wide.]

Anyway, the change is a very minor one.  We were including the four bytes
of the header in the length count and, as a result, sending four bytes of
garbage at the end of the SESSION REQUEST packet.

Small fix in function cli_session_request().


Revisions:
cliconnect.c		1.71 => 1.71.2.1
	http://www.samba.org/cgi-bin/cvsweb/samba/source/libsmb/cliconnect.c?r1=1.71&r2=1.71.2.1




More information about the samba-cvs mailing list