-
Notifications
You must be signed in to change notification settings - Fork 14k
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
Support rhosts in auxiliary module #11176
Conversation
The sanity test failures here are genuine. This PR breaks From the sanity test:
From a local test on macOS as well:
|
6093d1e
to
d2e01ab
Compare
@jmartin-r7 Thanks, fixed that. |
d2e01ab
to
96173c1
Compare
I think this is a serious issue, could someone in R7 pick this up? It also occurs in the latest msf5 release. |
Thanks for the ping @Green-m, taking a look. |
Aside from the inability to hit 'Ctrl^C' to cancel the module run, this worked as advertised. Do you want me to land it as-is, or would you like to look at what we need to do to allow canceling a module run across a subnet from the console? |
@busterb Thanks for picking this up! I have updated the code for cancel the module by 'Ctrl^C' |
The |
@msjenkins-r7 test this please. @Green-m, Sanity test looks like one of the 4 sessions did not connect, re-testing now. |
Thanks, looking good. |
Release NotesThis adds RHOSTS multiple-host targeting support to all auxiliary modules. It also corrects a bug targeting a single host from an exploit module using the syntax IP/32. |
This fixes a couple of bugs in rapid7#11176: - RHOSTS is not always used in Aux modules, don't enforce it unless it's used. - Be sure to pass the action to the run stub, since it's not a standard option.
For now, the
rhosts
option is not usable in auxiliary modules except scanners, it would throw an error likeSocketError getaddrinfo
.And in exploit module, same error occurred when
rhosts
set to8.8.8.8/32
.Fix #10693
Related: #9246
Steps to reproduce
Auxiliary module
exploit module
After fix
Auxiliary module
exploit module