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
base: master
from

Conversation

Projects
None yet
2 participants
@sethjackson

sethjackson commented Mar 10, 2017

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

This comment has been minimized.

dmiller-nmap commented Mar 10, 2017

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

This comment has been minimized.

sethjackson commented Mar 10, 2017

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

@sethjackson sethjackson deleted the sethjackson:cve-2017-5638 branch Mar 13, 2017

@dmiller-nmap

This comment has been minimized.

dmiller-nmap commented Mar 13, 2017

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

@sethjackson

This comment has been minimized.

sethjackson commented Mar 13, 2017

No worries. :)

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