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

Option to enable CWE78 scan #460

Open
ElDavoo opened this issue Apr 18, 2024 · 4 comments
Open

Option to enable CWE78 scan #460

ElDavoo opened this issue Apr 18, 2024 · 4 comments

Comments

@ElDavoo
Copy link

ElDavoo commented Apr 18, 2024

Hello,
I'd like to scan for everything that is supported, including CWE78, but I should specify all the supported CWEs with -p .
Is it possible to add options like -pp (?) to specify checks to be enabled? Or something similar?
Thank you!

@Enkelmann
Copy link
Collaborator

Well, we disabled the CWE-78 check on a standard run for a reason: It was leading to runtime explosion on a lot of binaries when enabled. Since then a lot has changed and we probably need to re-evaluate how the check is performing right now, but these problems probably still exist. If you want to use the check then my recommendation would be to run the cwe_checker twice: First with a standard run and then only with the CWE-78 check enabled. This way you quickly see if the CWE-78 check does run into runtime problems for your binary.

In general we disable checks for the standard run only if we think they are too unstable to be used regularly. And that is also the reason why we do not want to provide a simple shortcut to running these checks along all other checks.

@ElDavoo
Copy link
Author

ElDavoo commented Apr 21, 2024

First with a standard run and then only with the CWE-78 check enabled.

Thing is, I'm using cwe_checker in FACT

@Enkelmann
Copy link
Collaborator

FACT aborts the cwe_checker when a timeout is reached. And if that happens you do not get any results for any of the cwe_checker checks for the corresponding binary. So enabling the CWE-78 check for standard runs there is also a bad idea.

You could duplicate the cwe_checker plugin for FACT and just change the command line parameters for the duplicated plugin if you want to use my solution of running the cwe_checker twice for each binary. Alternatively, you could wait until we overhaul the CWE-78 check (although I cannot say when this will happen).

@ElDavoo
Copy link
Author

ElDavoo commented Apr 23, 2024

You could duplicate the cwe_checker plugin for FACT and just change the command line parameters for the duplicated plugin if you want to use my solution of running the cwe_checker twice for each binary.

Ok, I did that. Thanks for the input!

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

No branches or pull requests

2 participants