Merge branch 'nil_res_bug_fixes' of https://github.com/wchen-r7/metasploit-framework into wchen-r7-nil_res_bug_fixes

Author: jvazquez-r7

modules/auxiliary/admin/officescan/tmlisten_traversal.rb

@@ -51,6 +51,11 @@ def run_host(target_host)
 				'method'  => 'GET',
 			}, 20)
 
+		if not res
+			print_error("No response from server")
+			return
+		end
+
 		http_fingerprint({ :response => res })
 
 		if (res.code >= 200)

modules/auxiliary/dos/http/sonicwall_ssl_format.rb

@@ -58,7 +58,7 @@ def run
 				'uri' => datastore['URI'] + fmt,
 			})
 
-			if res.code == 200
+			if res and res.code == 200
 				res.body.scan(/\<td class\=\"loginError\"\>(.+)XX/ism)
 				print_status("Information leaked: #{$1}")
 			end

modules/auxiliary/scanner/http/ektron_cms400net.rb

@@ -67,7 +67,7 @@ def run_host(ip)
 
 			#Check for HTTP 200 response.
 			#Numerous versions and configs make if difficult to further fingerprint.
-			if (res.code == 200)
+			if (res and res.code == 200)
 				print_status("Ektron CMS400.NET install found at #{target_url}  [HTTP 200]")
 
 				#Gather __VIEWSTATE and __EVENTVALIDATION from HTTP response.

modules/auxiliary/scanner/http/sap_businessobjects_user_brute.rb

@@ -89,8 +89,8 @@ def enum_user(user='administrator', pass='pass')
 						'Content-Type'  => 'text/xml; charset=UTF-8',
 					}
 			}, 45)
-			return :abort if (res.code == 404)
-			success = true if(res.body.match(/SessionInfo/i))
+			return :abort if (!res or (res and res.code == 404))
+			success = true if(res and res.body.match(/SessionInfo/i))
 			success
 
 		rescue ::Rex::ConnectionError

modules/auxiliary/scanner/http/sap_businessobjects_user_brute_web.rb

@@ -75,7 +75,7 @@ def enum_user(user, pass)
 								'Accept-Encoding' => "gzip,deflate",
 							},
 			}, 45)
-			return :abort if (res.code != 200)
+			return :abort if (!res or (res and res.code != 200))
 			if(res.body.match(/Account Information/i))
 				success = false
 			else

modules/auxiliary/scanner/http/sap_businessobjects_user_enum.rb

@@ -93,8 +93,8 @@ def enum_user(user='administrator', pass='invalid-sap-password-0d03b389-b7a1-4ec
 			}, 45)
 
 			if res
-				return :abort if (res.code == 404)
-				success = true if(res.body.match(/Invalid password/i))
+				return :abort if (!res or (res and res.code == 404))
+				success = true if(res and res.body.match(/Invalid password/i))
 				success
 			else
 				vprint_error("[SAP BusinessObjects] No response")

modules/auxiliary/scanner/sap/sap_mgmt_con_brute_login.rb

@@ -126,6 +126,8 @@ def enum_user(user, pass)
 					}
 			}, 45)
 
+			return if not res
+
 			if (res.code != 500 and res.code != 200)
 				return
 			else

modules/exploits/unix/webapp/dogfood_spell_exec.rb

@@ -70,7 +70,7 @@ def check
 				'uri' => datastore['URIPATH'],
 			}, 1)
 
-		if (res.body =~ /Spell Check complete/)
+		if (res and res.body =~ /Spell Check complete/)
 			return Exploit::CheckCode::Detected
 		end
 		return Exploit::CheckCode::Safe