python.lang.security.audit.dangerous-code-run-audit.dangerous-interactive-code-run-audit

Author
unknown
Download Count*
License
Found dynamic content inside InteractiveConsole/InteractiveInterpreter method. This is dangerous if external data can reach this function call because it allows a malicious actor to run arbitrary Python code. Ensure no external data reaches here.
Run Locally
Run in CI
Defintion
rules:
- id: dangerous-interactive-code-run-audit
patterns:
- pattern-either:
- pattern: |
$X.push($PAYLOAD,...)
- pattern: |
$X.runsource($PAYLOAD,...)
- pattern: |
$X.runcode(code.compile_command($PAYLOAD),...)
- pattern: |
$PL = code.compile_command($PAYLOAD,...)
...
$X.runcode($PL,...)
- pattern-either:
- pattern-inside: |
$X = code.InteractiveConsole(...)
...
- pattern-inside: |
$X = code.InteractiveInterpreter(...)
...
- pattern-not: |
$X.push("...",...)
- pattern-not: |
$X.runsource("...",...)
- pattern-not: |
$X.runcode(code.compile_command("..."),...)
- pattern-not: |
$PL = code.compile_command("...",...)
...
$X.runcode($PL,...)
message: Found dynamic content inside InteractiveConsole/InteractiveInterpreter
method. This is dangerous if external data can reach this function call
because it allows a malicious actor to run arbitrary Python code. Ensure
no external data reaches here.
metadata:
cwe:
- "CWE-95: Improper Neutralization of Directives in Dynamically
Evaluated Code ('Eval Injection')"
owasp:
- A03:2021 - Injection
references:
- https://semgrep.dev/docs/cheat-sheets/python-command-injection/
category: security
technology:
- python
confidence: LOW
subcategory:
- audit
likelihood: LOW
impact: HIGH
license: Commons Clause License Condition v1.0[LGPL-2.1-only]
severity: WARNING
languages:
- python
Examples
dangerous-code-run-audit.py
import code
def run_payload1(payload: str) -> None:
console = code.InteractiveConsole()
# ruleid: dangerous-interactive-code-run-audit
console.push(payload)
def run_payload2(payload: str) -> None:
inperpreter = code.InteractiveInterpreter()
# ruleid: dangerous-interactive-code-run-audit
inperpreter.runcode(code.compile_command(payload))
def run_payload3(payload: str) -> None:
inperpreter = code.InteractiveInterpreter()
# ruleid: dangerous-interactive-code-run-audit
pl = code.compile_command(payload)
inperpreter.runcode(pl)
def run_payload4(payload: str) -> None:
inperpreter = code.InteractiveInterpreter()
# ruleid: dangerous-interactive-code-run-audit
inperpreter.runsource(payload)
def ok1() -> None:
console = code.InteractiveConsole()
console.push('print(123)')
def ok2() -> None:
inperpreter = code.InteractiveInterpreter()
inperpreter.runcode(code.compile_command('print(123)'))
def ok3() -> None:
inperpreter = code.InteractiveInterpreter()
pl = code.compile_command('print(123)')
inperpreter.runcode(pl)
def ok4() -> None:
inperpreter = code.InteractiveInterpreter()
inperpreter.runsource('print(123)')
Short Link: https://sg.run/WGe4