From 5f1ff38293cc41e235d92aa6ac19a3c8ef1146e5 Mon Sep 17 00:00:00 2001
From: Martin Sutovsky <martin_sutovsky@rapid7.com>
Date: Tue, 8 Jul 2025 07:27:32 +0200
Subject: [PATCH] Adds check for version negotiation response

---
 lib/rex/proto/smb/simple_client.rb | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/lib/rex/proto/smb/simple_client.rb b/lib/rex/proto/smb/simple_client.rb
index 04507cd75a2d7..0ad171a28b3eb 100644
--- a/lib/rex/proto/smb/simple_client.rb
+++ b/lib/rex/proto/smb/simple_client.rb
@@ -85,8 +85,12 @@ def login(name = '', user = '', pass = '', domain = '',
   
         dlog("SMB version(s) to negotiate: #{self.versions}")
         ok = self.client.negotiate
+        if ok.blank?
+          dlog("Failed to negotiate SMB version")
+          return false
+        end
         dlog("Negotiated SMB version: SMB#{negotiated_smb_version}")
-  
+
         if self.client.is_a?(RubySMB::Client)
           self.server_max_buffer_size = self.client.server_max_buffer_size
         else