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

Openvas scan bug #589

Closed
alirezakv opened this issue Oct 20, 2017 · 3 comments
Closed

Openvas scan bug #589

alirezakv opened this issue Oct 20, 2017 · 3 comments

Comments

@alirezakv
Copy link

The scan file for OpenVAS has a bug that occurs when there is only one target defined in OpenVAS which will cause the test to fail. if only one target is defined then the data structure returned by OpenVAS changes. This prevents scan execution.
The patch is attached here.

bug-openvas_scan_with_one_target.patch.txt
@alirezakv
Copy link
Author

@seccubus Could you look into this and #588 to include them in next build? I also included the patch files.

@Ar0xA
Copy link

Ar0xA commented Nov 18, 2017 •
edited

Hi @alirezakv, Just a suggestion but I know from experience that @seccubus prefers a pull request for changes to the code since this is easier for testing and merging.

If you're unable to make a PR from these patches personally, I can do it for you. Let me know.

@alirezakv
Copy link
Author

Hi @Ar0xA
Thank you for your tip.
I will do that.

alirezakv pushed a commit to alirezakv/Seccubus that referenced this issue Nov 27, 2017
[seccubus#589] Fix OpenVAS scan execution bug with only 1 target defined
8c0feac
@alirezakv alirezakv mentioned this issue Nov 27, 2017
Merged
MrSeccubus added a commit that referenced this issue Dec 7, 2017
@MrSeccubus
Merge pull request #621 from schubergphilis/588-and-589
17f39a0 
Closes #588 and #589
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Ar0xA @alirezakv