Land rapid7#3538, SAP fix from @jvazquez-r7

This looks good to me, the whole print statement is enclosed in a check
for results.

Author: Tod Beardsley

modules/auxiliary/scanner/sap/sap_mgmt_con_instanceproperties.rb

@@ -234,12 +234,14 @@ def enum_instance(rhost)
       if webmethods
         webmethods_output = [] # create empty webmethods array
         webmethods_arr = webmethods.split(",")
-        print_good("#{rhost}:#{rport} [SAP] Unprotected Webmethods :::")
         webmethods_arr.each do | webm |
           # Only add webmethods not found in protectedweb_arr
-          webmethods_output << webm if not protectedweb_arr.include?(webm)
+          webmethods_output << webm unless protectedweb_arr && protectedweb_arr.include?(webm)
+        end
+        if webmethods_output
+          print_good("#{rhost}:#{rport} [SAP] Unprotected Webmethods :::")
+          print_status("#{webmethods_output.join(',')}")
         end
-        print_status("#{webmethods_output.join(',')}") if webmethods_output
         report_note(:host => rhost,
               :proto => 'tcp',
               :port => rport,