Merge pull request rapid7#8261 from dmaloney-r7/bug/disable_smb_login_nbss

Bug/disable smb login nbss

Author: dmaloney-r7

lib/metasploit/framework/login_scanner/smb.rb

@@ -30,7 +30,7 @@ module AccessLevels
 
         CAN_GET_SESSION      = true
         DEFAULT_REALM        = 'WORKSTATION'
-        LIKELY_PORTS         = [ 139, 445 ]
+        LIKELY_PORTS         = [ 445 ]
         LIKELY_SERVICE_NAMES = [ "smb" ]
         PRIVATE_TYPES        = [ :password, :ntlm_hash ]
         REALM_KEY            = Metasploit::Model::Realm::Key::ACTIVE_DIRECTORY_DOMAIN

spec/lib/metasploit/framework/login_scanner_spec.rb

@@ -24,16 +24,16 @@
     it { is_expected.not_to include Metasploit::Framework::LoginScanner::HTTP }
   end
 
-  [ 139, 445 ].each do |foo|
-    context "with port #{foo}" do
-      let(:port) { foo }
 
-      it { is_expected.to include Metasploit::Framework::LoginScanner::SMB }
-      it { is_expected.not_to include Metasploit::Framework::LoginScanner::HTTP }
-      it { is_expected.not_to include Metasploit::Framework::LoginScanner::VNC }
-    end
+  context "with port 445" do
+    let(:port) { 445 }
+
+    it { is_expected.to include Metasploit::Framework::LoginScanner::SMB }
+    it { is_expected.not_to include Metasploit::Framework::LoginScanner::HTTP }
+    it { is_expected.not_to include Metasploit::Framework::LoginScanner::VNC }
   end
 
+
   context "with name 'http'" do
     let(:name) { 'http' }