svn commit: samba r24320 - in branches/SAMBA_3_2/source/smbd: .

vlendec at samba.org vlendec at samba.org
Fri Aug 10 21:34:57 GMT 2007


Author: vlendec
Date: 2007-08-10 21:34:55 +0000 (Fri, 10 Aug 2007)
New Revision: 24320

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

Log:
Fix the wct checks in reply_trans2 and the reply_trans calls

No idea what I've been smoking here.

Thanks to Stefan for closely looking

Modified:
   branches/SAMBA_3_2/source/smbd/ipc.c
   branches/SAMBA_3_2/source/smbd/trans2.c


Changeset:
Modified: branches/SAMBA_3_2/source/smbd/ipc.c
===================================================================
--- branches/SAMBA_3_2/source/smbd/ipc.c	2007-08-10 21:33:58 UTC (rev 24319)
+++ branches/SAMBA_3_2/source/smbd/ipc.c	2007-08-10 21:34:55 UTC (rev 24320)
@@ -488,7 +488,7 @@
 
 	START_PROFILE(SMBtrans);
 
-	if (SVAL(req->inbuf, smb_wct) < 10) {
+	if (req->wct < 14) {
 		reply_nterror(req, NT_STATUS_INVALID_PARAMETER);
 		END_PROFILE(SMBtrans);
 		return;
@@ -667,7 +667,7 @@
 
 	show_msg((char *)req->inbuf);
 
-	if (req->wct < 10) {
+	if (req->wct < 8) {
 		reply_nterror(req, NT_STATUS_INVALID_PARAMETER);
 		END_PROFILE(SMBtranss);
 		return;

Modified: branches/SAMBA_3_2/source/smbd/trans2.c
===================================================================
--- branches/SAMBA_3_2/source/smbd/trans2.c	2007-08-10 21:33:58 UTC (rev 24319)
+++ branches/SAMBA_3_2/source/smbd/trans2.c	2007-08-10 21:34:55 UTC (rev 24320)
@@ -7028,7 +7028,7 @@
 
 	START_PROFILE(SMBtrans2);
 
-	if (req->wct < 8) {
+	if (req->wct < 14) {
 		reply_nterror(req, NT_STATUS_INVALID_PARAMETER);
 		END_PROFILE(SMBtrans2);
 		return;



More information about the samba-cvs mailing list