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

tridge at samba.org tridge at samba.org
Sat Nov 13 07:22:14 GMT 2004


Author: tridge
Date: 2004-11-13 07:22:14 +0000 (Sat, 13 Nov 2004)
New Revision: 3720

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

Log:
fixed the pulling of zero length ucs2 strings in smbd. I found this
bug using ifstest.exe, which is a IFS (Installable File System) test
suite. With this fix I was pleasantly surprised to find that Samba4
passes many (maybe even most?) of the tests in ifstest.

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


Changeset:
Modified: branches/SAMBA_4_0/source/smb_server/request.c
===================================================================
--- branches/SAMBA_4_0/source/smb_server/request.c	2004-11-13 06:49:13 UTC (rev 3719)
+++ branches/SAMBA_4_0/source/smb_server/request.c	2004-11-13 07:22:14 UTC (rev 3720)
@@ -489,6 +489,11 @@
 	}
 
 	src_len2 = utf16_len_n(src, src_len);
+	if (src_len2 == 0) {
+		*dest = talloc_strdup(req, "");
+		return src_len2 + alignment;
+	}
+
 	ret = convert_string_talloc(req, CH_UTF16, CH_UNIX, src, src_len2, (void **)&dest2);
 
 	if (ret == -1) {



More information about the samba-cvs mailing list