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

Update to display vuln obj to user for CVE2006-3392 #1220

Closed
wants to merge 1 commit into from
Closed

Update to display vuln obj to user for CVE2006-3392 #1220

wants to merge 1 commit into from

Conversation

rewardone
Copy link

Previously, the function would return vuln_report:make_output(detection_session.body).

The expected output is that the nmap scan finishes and shows the "vuln" structure (title, state, IDs, description) if the instance is vulnerable.

This was not the case when run against a vulnerable instance (pwnOs1 vulnerable VM). The nmap script would finish and not display anything to the user unless --script-trace or -d flags were set.

After updating make_output to make_output(vuln), the finished nmap scan correctly displays the vuln structure after completion.

A further enhancement could be made to add detection_session.body to the vuln obj.

Previously, the function would return vuln_report:make_output(detection_session.body). 
The expected output is that the nmap scan finishes and shows the "vuln" structure (title, state, IDs, description) if the instance is vulnerable.
This was not the case when run against a vulnerable instance (pwnOs1 vulnerable VM). The nmap script would finish and not display anything to the user unless --script-trace or -d flags were set.

After updating make_output to make_output(vuln), the finished nmap scan correctly displays the vuln structure after completion.

A further enhancement could be made to add detection_session.body to the vuln obj.
@cldrn cldrn self-assigned this May 25, 2018
@cldrn cldrn added the NSE label May 25, 2018
@nmap-bot nmap-bot closed this in fc3062b May 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants