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

Fixed unwanted execution and wrong database path for p10 analysis with CodeQL #83

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

federicozappone
Copy link

Fixed two small issues that prevented me from correctly running the CodeQL tool with or without the P10 checks.

  1. When running the CodeQL static analysis the Scrub tool would execute the P10 checks even if the "CODEQL_P10_ANALYSIS" parameter in the scrub.cfg file is set to False.
  2. The same template file for CodeQL would use the wrong database path when running the "codeql database analyze" command for the P10 checks resulting in an exception.

@federicozappone federicozappone changed the title fixed unwanted execution and wrong database path for p10 analysis Fixed unwanted execution and wrong database path for p10 analysis with CodeQL Jan 17, 2023
federicozappone and others added 6 commits January 21, 2023 19:50
Added the new flags for the CodeQL open-source coding standards path and the AUTOSAR analysis flag.
Also modified the template config to reflect the standard CodeQL paths organization specified in their tutorial.
The CodeQL coding standard repo containing rules for the AUTOSAR, MISRA, and CERT standards is available here: https://github.com/github/codeql-coding-standards
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.

1 participant