Small tweaks based on @bcoles feedback. Thanks!

Author: hdm

modules/exploits/linux/http/goahead_ldpreload.rb

@@ -174,7 +174,7 @@ def cycle_possible_payloads
       end
     end
 
-    # Remove any duplicates that mau have snuck in
+    # Remove any duplicates that may have snuck in
     template_list.uniq!
 
     # Cycle through each top-level platform we know about
@@ -185,7 +185,7 @@ def cycle_possible_payloads
 
 
         wrapper_path = ::File.join(template_base, "goahead-cgi-#{template_type}-#{t_plat}-#{t_arch}.so.gz")
-        if ! ::File.exists?(wrapper_path)
+        unless ::File.exist?(wrapper_path)
           raise RuntimeError.new("Missing executable template at #{wrapper_path}")
         end
 
@@ -217,7 +217,7 @@ def cycle_possible_payloads
         yield(data)
 
         # Introduce a small delay for the payload to stage
-        sleep(0.50)
+        Rex.sleep(0.50)
 
         # Short-circuit once we have a session
         return if session_created?
@@ -378,14 +378,14 @@ def is_cgi_exploitable?(uri)
       vprint_status("Request for #{uri} did not return a response")
     end
 
-    !!(res && res.body && res.body.to_s.index("LD_DEBUG_OUTPUT"))
+    !!(res && res.body && res.body.to_s.include?("LD_DEBUG_OUTPUT"))
   end
 
   # This sometimes determines if the CGI module is enabled, but doesn't seem
   # to return the error to the client in newer versions. Unused for now.
   def is_cgi_enabled?
     return true
     res = send_request_cgi({'uri' => "/cgi-bin"})
-    !!(res && res.body && res.body.to_s.index("Missing CGI name"))
+    !!(res && res.body && res.body.to_s.include?("Missing CGI name"))
   end
 end