Land #4161, "stop" NilClass fix

rapid7 · Nov 10, 2014 · 0e772cc · 0e772cc · wchen-r7 · Nov 10, 2014
2 parents bc55293 + 8f3b1e7
commit 0e772cc
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/lib/rex/exploitation/cmdstager/tftp.rb b/lib/rex/exploitation/cmdstager/tftp.rb
@@ -31,14 +31,14 @@ def initialize(exe)
   end
 
   def setup(mod)
-    tftp = Rex::Proto::TFTP::Server.new
-    tftp.register_file(Rex::Text.rand_text_alphanumeric(8), exe)
-    tftp.start
-    mod.add_socket(tftp) # Hating myself for doing it... but it's just a first demo
+    self.tftp = Rex::Proto::TFTP::Server.new
+    self.tftp.register_file(Rex::Text.rand_text_alphanumeric(8), exe)
+    self.tftp.start
+    mod.add_socket(self.tftp) # Hating myself for doing it... but it's just a first demo
   end
 
   def teardown(mod = nil)
-    tftp.stop
+    self.tftp.stop
   end
 
   #

diff --git a/modules/exploits/multi/http/struts_code_exec.rb b/modules/exploits/multi/http/struts_code_exec.rb
@@ -10,6 +10,7 @@ class Metasploit3 < Msf::Exploit::Remote
 
   include Msf::Exploit::CmdStager
   include Msf::Exploit::Remote::HttpClient
+  include Msf::Exploit::EXE
 
   def initialize(info = {})
     super(update_info(info,
@@ -91,7 +92,7 @@ def execute_command(cmd, opts = {})
   def windows_stager
     print_status("Sending request to #{datastore['RHOST']}:#{datastore['RPORT']}")
     execute_cmdstager({ :temp => '.' })
-    @payload_exe = payload_exe
+    @payload_exe = generate_payload_exe
 
     print_status("Attempting to execute the payload...")
     execute_command(@payload_exe)