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

Add script for CVE-2017-5638. #740

Closed
wants to merge 1 commit into from
Closed

Add script for CVE-2017-5638. #740

wants to merge 1 commit into from

Conversation

sethjackson
Copy link

Add a NSE script for CVE-2017-5638.

The script adds a header on the server side via the RCE and checks that the header value is what we set it to.

More info here.

@dmiller-nmap
Copy link

Awesome! Very clean script. I had to add a missing require "string", which you could have caught with our code check tool and git hook, but otherwise this worked great. Thanks!

@sethjackson
Copy link
Author

Ah whoops! Sorry about that. I didn't see an errors when running locally so I missed that.

@sethjackson sethjackson deleted the cve-2017-5638 branch March 13, 2017 13:32
@dmiller-nmap
Copy link

oops, forgot to link this to the commit: 3140dbe

@sethjackson
Copy link
Author

No worries. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants