Cleanup based on PR rapid7#1008 feedback

Author: HD Moore

modules/auxiliary/scanner/scada/digi_realport_serialport_scan.rb

@@ -57,7 +57,9 @@ def run_host(target_host)
 
 		return unless realport_connect
 
-		vprint_status("#{target_host}:#{rport} is running #{@realport_name} ( ports: #{@ports} )")
+		info = "#{@realport_name} ( ports: #{@realport_port_count} )"
+		vprint_status("#{target_host}:#{rport} is running #{info}")
+		report_service(:host => rhost, :port => rport, :name => "realport", :info => info)
 
 		1.upto(@realport_port_count) do |pnum|
 			unless test_ports.include?('ALL') or test_ports.include?(pnum.to_s)
@@ -71,6 +73,15 @@ def run_host(target_host)
 				res = realport_recv_banner(pnum - 1, datastore['BANNER_TIMEOUT'])
 				if res and res.length > 0
 					print_status("#{target_host}:#{rport} [port #{pnum} @ #{baud}bps] #{res.inspect}")
+					report_note(
+						:host   => target_host,
+						:proto  => 'tcp',
+						:port   => rport,
+						:type   => "realport.port#{pnum}.banner",
+						:data   => {:baud => baud, :banner => res},
+						:update => :unique_data
+					)
+
 				end
 				realport_close(pnum - 1)
 			end

modules/auxiliary/scanner/scada/digi_realport_version.rb

@@ -40,5 +40,6 @@ def run_host(target_host)
 			print_status("#{target_host}:#{rport} #{info}")
 			report_service(:host => rhost, :port => rport, :name => "realport", :info => info)
 		end
+		realport_disconnect
 	end
 end