requested change resolved, PR #20418

Author: H4k1l

modules/auxiliary/analyze/crack_aix.rb

@@ -25,9 +25,8 @@ def initialize
       'Actions' => [
         ['john', { 'Description' => 'Use John the Ripper' }],
         ['hashcat', { 'Description' => 'Use Hashcat' }],
-        ['auto', { 'Description' => 'Use either John the Ripper or Hashcat, if both are present, use Hashcat' }] 
       ],
-      'DefaultAction' => 'auto',
+      'DefaultAction' => 'john',
       'Notes' => {
         'Stability' => [CRASH_SAFE],
         'SideEffects' => [],
@@ -45,21 +44,16 @@ def initialize
 
   def show_command(cracker_instance)
     return unless datastore['ShowCommand']
-    
-    newaction = getaction()
 
-    if newaction == 'john'
+    if action.name == 'john'
       cmd = cracker_instance.john_crack_command
-    elsif newaction == 'hashcat'
+    elsif action.name == 'hashcat'
       cmd = cracker_instance.hashcat_crack_command
     end
     print_status("   Cracking Command: #{cmd.join(' ')}")
   end
 
   def check_results(passwords, results, hash_type, method)
-
-    newaction = getaction()
-
     passwords.each do |password_line|
       password_line.chomp!
       next if password_line.blank?
@@ -69,12 +63,12 @@ def check_results(passwords, results, hash_type, method)
       next unless fields.count >= 3
 
       cred = { 'hash_type' => hash_type, 'method' => method }
-      if newaction == 'john'
+      if action.name == 'john'
         cred['username'] = fields.shift
         cred['core_id'] = fields.pop
         4.times { fields.pop } # Get rid of extra :
         cred['password'] = fields.join(':') # Anything left must be the password. This accounts for passwords with semi-colons in it
-      elsif newaction == 'hashcat'
+      elsif action.name == 'hashcat'
         cred['core_id'] = fields.shift
         cred['hash'] = fields.shift
         cred['password'] = fields.join(':') # Anything left must be the password. This accounts for passwords with semi-colons in it
@@ -91,17 +85,14 @@ def check_results(passwords, results, hash_type, method)
   end
 
   def run
-
-    newaction = getaction()
-
     tbl = tbl = cracker_results_table
 
     hash_types_to_crack = ['descrypt']
     jobs_to_do = []
 
     # build our job list
     hash_types_to_crack.each do |hash_type|
-      job = hash_job(hash_type, newaction)
+      job = hash_job(hash_type, action.name)
       if job.nil?
         print_status("No #{hash_type} found to crack")
       else
@@ -119,7 +110,7 @@ def run
     # Inner array format: db_id, hash_type, username, password, method_of_crack
     results = []
 
-    cracker = new_password_cracker(newaction)
+    cracker = new_password_cracker(action.name)
 
     # generate our wordlist and close the file handle.  max length of DES is 8
     wordlist = wordlist_file(8)
@@ -145,7 +136,7 @@ def run
       cracker_instance = cracker.dup
       cracker_instance.format = format
 
-      if newaction == 'john'
+      if action.name == 'john'
         cracker_instance.fork = datastore['FORK']
       end
 
@@ -156,7 +147,7 @@ def run
       job['cred_ids_left_to_crack'] = job['cred_ids_left_to_crack'] - results.map { |i| i[0].to_i } # remove cracked hashes from the hash list
       next if job['cred_ids_left_to_crack'].empty?
 
-      if newaction == 'john'
+      if action.name == 'john'
         print_status "Cracking #{format} hashes in single mode..."
         cracker_instance.mode_single(wordlist.path)
         show_command cracker_instance
@@ -198,7 +189,7 @@ def run
       print_status "Cracking #{format} hashes in wordlist mode..."
       cracker_instance.mode_wordlist(wordlist.path)
       # Turn on KoreLogic rules if the user asked for it
-      if newaction == 'john' && datastore['KORELOGIC']
+      if action.name == 'john' && datastore['KORELOGIC']
         cracker_instance.rules = 'KoreLogicRules'
         print_status 'Applying KoreLogic ruleset...'
       end
@@ -222,24 +213,4 @@ def run
       end
     end
   end
-
-  def getaction
-    newaction = action.name
-    if action.name == 'auto'
-      path = Rex::FileUtils.find_full_path('hashcat') ||
-      Rex::FileUtils.find_full_path('hashcat.exe')
-      if path
-        newaction = 'hashcat'
-      else
-        path = Rex::FileUtils.find_full_path('john') ||
-        Rex::FileUtils.find_full_path('john.exe')
-        if path
-          newaction = 'john'
-        else
-          raise PasswordCrackerNotFoundError, 'No suitable john/hashcat binary was found on the system'
-        end
-      end
-    end
-    return newaction
-  end
 end