From 67f9577de9303de7a4c49b0c8ba3ff44bbaabf0d Mon Sep 17 00:00:00 2001
From: ron <ron@e0a8ed71-7df4-0310-8962-fdc924857419>
Date: Wed, 24 Dec 2008 20:36:30 +0000
Subject: [PATCH] No longer erroring if a server leaves out the 'domain'
 section of packets (some configurations of Samba seem to do this)

---
 nselib/smb.lua | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/nselib/smb.lua b/nselib/smb.lua
index a9c82fd5c..2d9afe431 100644
--- a/nselib/smb.lua
+++ b/nselib/smb.lua
@@ -1030,21 +1030,16 @@ function negotiate_protocol(smb)
 		return false, "SMB: ERROR: Ran off the end of SMB packet; likely due to server truncation [12]"
 	end
 
-	-- Get the domain as a Unicode string
 	local ch, dummy
 	domain = ""
 	server = ""
 
+	-- Get the domain as a Unicode string
+	-- Note: This can be null, some configurations of Samba leave this off
 	pos, ch, dummy = bin.unpack("<CC", data, pos)
-	if(dummy == nil) then
-		return false, "SMB: ERROR: Ran off the end of SMB packet; likely due to server truncation [13]"
-	end
 	while ch ~= nil and ch ~= 0 do
 		domain = domain .. string.char(ch)
 		pos, ch, dummy = bin.unpack("<CC", data, pos)
-		if(dummy == nil) then
-			return false, "SMB: ERROR: Ran off the end of SMB packet; likely due to server truncation [14]"
-		end
 	end
 
 	-- Get the server name as a Unicode string