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

Expand detection information of results #1182

Merged

Conversation

timopollmeier
Copy link
Member

@timopollmeier timopollmeier commented Jul 9, 2020

This adds a "path" column and the use of additional host details to results to better handle cases where products were detected in multiple locations on one host.
For example multiple versions of a product are installed and a vulnerability is reported for a specific one.

Checklist:

This new column allows better identification of where products were
found locally on target hosts and consequently better handling of
multiple instances of products.
The OID will now be determined inside result_detection_reference
as the host detail to be used will depend on other detection data.
The column with the detected_by detail is no longer used and if possible
the path should be used to determine the detection details.
The sql() call contained a stray comma, the output was not initialized
and some uses were not dereferenced.
This allows using new "detected_by@" host details that specify the NVT
used to detect a vulnerability at a specific path or port.
@timopollmeier timopollmeier changed the title Expand detection information of result Expand detection information of results Jul 9, 2020
@timopollmeier timopollmeier changed the base branch from master to gvmd-20.08 July 9, 2020 16:08
@timopollmeier timopollmeier marked this pull request as ready for review July 9, 2020 16:11
@mattmundell mattmundell merged commit 0a32935 into greenbone:gvmd-20.08 Jul 13, 2020
@timopollmeier timopollmeier deleted the result-path-field-20.08 branch October 15, 2021 10:11
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.

2 participants