Land rapid7#2089, smb last_filename can be nil

Author: egypt

lib/rex/proto/smb/client.rb

@@ -1881,7 +1881,7 @@ def find_first(path)
 						'C'+	# Short File Name Length
 						'C' 	# Reserved
 					)
-					name = resp_data[didx + 70 + 24, info[15]].sub!(/\x00+$/, '')
+					name = resp_data[didx + 70 + 24, info[15]].sub(/\x00+$/, '')
 					files[name] =
 					{
 						'type' => ((info[14] & 0x10)==0x10) ? 'D' : 'F',
@@ -1916,7 +1916,7 @@ def find_next(sid, resume_key, last_filename)
 			260, # Level of interest
 			resume_key,   # Resume key from previous (Last name offset)
 			6,   # Close search if end of search
-		].pack('vvvVv') + last_filename + "\x00" # Last filename returned from find_first or find_next
+		].pack('vvvVv') + last_filename.to_s + "\x00" # Last filename returned from find_first or find_next
 		resp = trans2(CONST::TRANS2_FIND_NEXT2, parm, '')
 		return resp # Returns the FIND_NEXT2 response packet for parsing by the find_first function
 	end