-
Notifications
You must be signed in to change notification settings - Fork 14.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adds support for resolving multiple host ips #18499
base: master
Are you sure you want to change the base?
Adds support for resolving multiple host ips #18499
Conversation
It'd be good to update the test notes to verify that an ipv6 target works as expected |
Will attic for now until we can pick this up again in the new year when we've got the other priorities out for this year 👍 |
Thanks for your contribution to Metasploit Framework! We've looked at this pull request, and we agree that it seems like a good addition to Metasploit, but it looks like it is not quite ready to land. We've labeled it What does this generally mean? It could be one or more of several things:
We would love to land this pull request when it's ready. If you have a chance to address all comments, we would be happy to reopen and discuss how to merge this! |
a81f513
to
22529d6
Compare
22529d6
to
5a005b2
Compare
This PR is in conjunction with a PR in metasploit-payloads.
This PR updates the
resolve_host
andresolve_hosts
to now support resolving multiple host IPs.The changes support backwards compatibility for old Meterpreter sessions by checking for TLV types off
TLV_TYPE_IP
that follows the existing logic, as well as the newTLV_TYPE_RESOLVE_HOST_ENTRY
TLV.Verification
Needs to be tested in tandem with the metasploit-payloads PR.
resolve_host
msfconsole
use php/meterpreter/reverse_tcp
irb -e "framework.sessions.values.last.net.resolve.resolve_host 'rapid7.com'"
resolve_hosts
msfconsole
use php/meterpreter/reverse_tcp
irb -e "framework.sessions.values.last.net.resolve.resolve_hosts ['rapid7.com', 'google.com']"