Last minute linting

Author: jheysel-r7

modules/auxiliary/admin/ldap/bad_successor.rb

@@ -80,19 +80,14 @@ def windows_version_vulnerable?
   def validate
     errors = {}
 
-    unless %w[ auto ntlm plaintext]
-      # if AUTO changes in the future to not require a password, we'll need to reevaluate this
-      errors['LDAP::Auth'] = 'Only password-based LDAP authentication methods are supported with this exploit.'
-    end
-
     case action.name
     when 'GET_TICKET'
-      if %w[ auto ntlm ].include?(datastore['LDAP::Auth']) && Net::NTLM.is_ntlm_hash?(datastore['LDAPPassword'].encode(::Encoding::UTF_16LE))
+      if %w[auto ntlm].include?(datastore['LDAP::Auth']) && Net::NTLM.is_ntlm_hash?(datastore['LDAPPassword'].encode(::Encoding::UTF_16LE))
         errors['LDAPPassword'] = 'The GET_TICKET action is incompatible with LDAP passwords that are NTLM hashes.'
       end
     end
 
-    raise Msf::OptionValidateError.new(errors) unless errors.empty?
+    raise Msf::OptionValidateError, errors unless errors.empty?
   end
 
   def check
@@ -234,7 +229,6 @@ def set_dmsa_attributes(dn, delegated_state, preceded_by_link)
   end
 
   def query_account(account_name)
-    account_name = datastore['DMSA_ACCOUNT_NAME']
     account_name += '$' unless account_name.ends_with?('$')
     entry = adds_get_object_by_samaccountname(@ldap, account_name)