scripts/test_demos.py
Consider possible security implications associated with subprocess module. Open
Open
import subprocess
- Exclude checks
subprocess call with shell=True identified, security issue. Open
Open
procout = subprocess.run(
cmd_line,
shell=True,
check=False,
env=environ,
- Exclude checks
Remove this commented out code. Open
Open
# stdout=subprocess.PIPE,
- Read upRead up
- Exclude checks
Programmers should not comment out code as it bloats programs and reduces readability.
Unused code should be deleted and can be retrieved from source control history if required.
See
- MISRA C:2004, 2.4 - Sections of code should not be "commented out".
- MISRA C++:2008, 2-7-2 - Sections of code shall not be "commented out" using C-style comments.
- MISRA C++:2008, 2-7-3 - Sections of code should not be "commented out" using C++ comments.
- MISRA C:2012, Dir. 4.4 - Sections of code should not be "commented out"