svn commit: samba r14526 - in branches/SAMBA_4_0/source/smb_server/smb: .

metze at samba.org metze at samba.org
Fri Mar 17 17:20:56 GMT 2006


Author: metze
Date: 2006-03-17 17:20:54 +0000 (Fri, 17 Mar 2006)
New Revision: 14526

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=14526

Log:
fix the size of the output buffer, we need 1 byte for the Word count
now smbsrv_setup_reply() and req_setup_chain_reply() do the same thing

tridge: can you please check if this is correct,
        I wonder why we don't got valgrind errors...
	as the over allocation is disabled currently

metze
Modified:
   branches/SAMBA_4_0/source/smb_server/smb/request.c


Changeset:
Modified: branches/SAMBA_4_0/source/smb_server/smb/request.c
===================================================================
--- branches/SAMBA_4_0/source/smb_server/smb/request.c	2006-03-17 17:20:24 UTC (rev 14525)
+++ branches/SAMBA_4_0/source/smb_server/smb/request.c	2006-03-17 17:20:54 UTC (rev 14526)
@@ -105,7 +105,7 @@
 		return;
 	}
 
-	req->out.size = NBT_HDR_SIZE + MIN_SMB_SIZE + wct*2 + buflen;
+	req->out.size = NBT_HDR_SIZE + MIN_SMB_SIZE + 1 + VWV(wct) + 2 + buflen;
 
 	/* over allocate by a small amount */
 	req->out.allocated = req->out.size + REQ_OVER_ALLOCATION; 



More information about the samba-cvs mailing list