fixed ddns_creds import issue, by using rhost and commenting why it needs to be used

bug/bundler_fix
Tyler Bennett 2015-12-04 15:10:02 -05:00
parent 6ce54f15ee
commit 4e0ab9b68f
1 changed files with 6 additions and 3 deletions

View File

@ -143,7 +143,7 @@ class Metasploit3 < Msf::Auxiliary
ddns_table << ["#{ddns_service}", "#{ddns_server}", "#{ddns_port}","#{ddns_domain}", "#{ddns_user}", "#{ddns_pass}"]
ddns_table.print
unless ddns_server.blank? && ddns_port.blank? && ddns_user.blank? && ddns_pass.blank?
report_ddns_cred(ddns_server, ddns_port, ddns_user, ddns_pass)
report_ddns_cred(rhost, ddns_port, ddns_user, ddns_pass)
end
end
end
@ -319,9 +319,12 @@ class Metasploit3 < Msf::Auxiliary
create_credential_login(login_data)
end
def report_ddns_cred(ddns_server, ddns_port, ddns_user, ddns_pass)
def report_ddns_cred(rhost, ddns_port, ddns_user, ddns_pass)
service_data = {
address: ddns_server,
# if a system returns a domain name for ddns_server this will fail to
# input into the db because the creds api expects an inet record. This has to be set to RHOST until issue #3968 is
# sorted in msf framework.
address: rhost,
port: ddns_port,
service_name: 'ddns settings',
protocol: 'tcp',