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

ms17_010_psexec: fix RHOST in "authenticating..." message #11267

Merged
merged 1 commit into from Jan 17, 2019

Conversation

Projects
None yet
4 participants
@cnotin
Copy link
Contributor

cnotin commented Jan 16, 2019

The "Authenticating..." message has an info missing:

Authenticating to as user 'user'...

It is due to a confusion between RHOST and RHOSTS.
This module uses RHOST as shown just below in the code:
eternal_pwn(datastore['RHOST'])

@wchen-r7

This comment has been minimized.

Copy link
Contributor

wchen-r7 commented Jan 16, 2019

Note: rspec failure resolved in #11269.

@cnotin

This comment has been minimized.

Copy link
Contributor Author

cnotin commented Jan 16, 2019

Well, I was surprised to see the build fail at first 😉

@wvu-r7 wvu-r7 added module bug labels Jan 17, 2019

@wvu-r7 wvu-r7 self-assigned this Jan 17, 2019

@wvu-r7 wvu-r7 merged commit 31a7b13 into rapid7:master Jan 17, 2019

3 checks passed

Metasploit Automation - Sanity Test Execution Successfully completed all tests.
Details
Metasploit Automation - Test Execution Successfully completed all tests.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

wvu-r7 added a commit that referenced this pull request Jan 17, 2019

@wvu-r7

This comment has been minimized.

Copy link
Contributor

wvu-r7 commented Jan 17, 2019

Release Notes

This fixes a missing RHOST in the authentication message for exploit/windows/smb/ms17_010_psexec.

msjenkins-r7 added a commit that referenced this pull request Jan 17, 2019

@wvu-r7

This comment has been minimized.

Copy link
Contributor

wvu-r7 commented Jan 17, 2019

Merci, @cnotin!

@gdavidson-r7 gdavidson-r7 added the rn-fix label Jan 23, 2019

@cnotin cnotin deleted the cnotin:patch-1 branch Feb 13, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment