Skip to content
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

Fix inconstancy - multi/handler #5527

Merged
merged 2 commits into from Jun 15, 2015
Merged

Fix inconstancy - multi/handler #5527

merged 2 commits into from Jun 15, 2015

Conversation

g0tmi1k
Copy link
Contributor

@g0tmi1k g0tmi1k commented Jun 12, 2015

Rather than it being a mixture of:

  • [*] Starting exploit multi handler
  • [*] Stopping multi/handler

Replace them all to be exploit/multi/handler.

Proof

resource (/tmp/msf.rc)> run -z
[*] Started reverse handler on 192.168.1.4:443 
...SNIP...
[*] Session ID 1 (192.168.1.4:443 -> 192.168.1.8:49236) processing AutoRunScript 'post/multi/manage/shell_to_meterpreter -o LPORT=4433'
[*] Upgrading session: 1
**[*] Starting exploit multi handler**
[*] Started reverse handler on 192.168.1.4:4433 
...SNIP...
**[*] Stopping multi/handler**
...SNIP...

@mubix
Copy link
Contributor

mubix commented Jun 13, 2015

Really cool, never noticed how many places it said different things for multi handler ;-)

@wvu wvu self-assigned this Jun 13, 2015
@wvu
Copy link
Contributor

wvu commented Jun 13, 2015

Great!

@wvu wvu merged commit 6dcc9b7 into rapid7:master Jun 15, 2015
wvu added a commit that referenced this pull request Jun 15, 2015
@g0tmi1k
Copy link
Contributor Author

g0tmi1k commented Jun 15, 2015

@mubix yet its been bugging me for ages =).

Thanks @wvu-r7. My OCD is a little happier now ;).

@g0tmi1k g0tmi1k deleted the inconstancy branch June 15, 2015 16:00
@wvu
Copy link
Contributor

wvu commented Jun 15, 2015

@g0tmi1k: I was looking at this myself but didn't bother fixing it. So thanks. :)

g0tmi1k added a commit to g0tmi1k/metasploit-framework that referenced this pull request Jun 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants