Skip to content

Commit 8cf10be

Browse files
author
Tod Beardsley
committed
Don't assume SSLv3 is set (kill FP+s)
1 parent 0b67efd commit 8cf10be

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

modules/auxiliary/scanner/http/http_sslv3.rb

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,8 +55,13 @@ def run_host(ip)
5555
res = send_request_raw({ 'uri' => '/', 'method' => 'GET' })
5656
fp = http_fingerprint(:response => res)
5757
if fp
58-
print_good("#{peer} #{fp} accepts SSLv3")
59-
report_poodle_vuln(ip)
58+
vprint_status("#{peer} connected and fingerprinted: #{fp}")
59+
# TODO: Interrogate the connection itself to see what version
60+
# was used. Where that actually lives is eluding me. :/
61+
if datastore['SSLVersion'] == 'SSL3'
62+
print_good("#{peer} accepts SSLv3")
63+
report_poodle_vuln(ip)
64+
end
6065
end
6166
rescue ::OpenSSL::SSL::SSLError => e
6267
ssl_version = e.message.match(/ state=([^\s]+)/)[1]

0 commit comments

Comments
 (0)