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.

Sign up for GitHub

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

Jump to bottom

Fix nil error when target not found #5632

Merged
merged 1 commit into from Jul 1, 2015
Merged

Fix nil error when target not found #5632

merged 1 commit into from Jul 1, 2015

Conversation

wvu
Copy link
Contributor

@wvu wvu commented Jun 29, 2015

Discovered in #5624.

  • Fire up exploit/windows/browser/mozilla_reduceright
  • Test with an unexpected user agent
  • See a 404
  • Don't see a nil error in msfconsole
  • Test with an expected user agent
  • See the exploit code

I'm using "NT 5.1 Firefox/3.6.16" as a valid user agent.

@wvu wvu mentioned this pull request Jun 29, 2015
Closed
@wvu wvu closed this Jun 29, 2015
@wvu wvu reopened this Jun 29, 2015
@wvu wvu closed this Jun 29, 2015
@wvu wvu reopened this Jun 29, 2015
MSadek-r7 added a commit to MSadek-r7/metasploit-framework that referenced this pull request Jul 1, 2015
@MSadek-r7
Land rapid7#5632, mozilla_reduceright nil fix
1c5abec
@MSadek-r7 MSadek-r7 merged commit 3632cc4 into rapid7:master Jul 1, 2015
@wvu wvu deleted the beug/nil branch July 1, 2015 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@MSadek-r7 MSadek-r7

Labels
bug module
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@wvu @MSadek-r7