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

profile photo of semgrepsemgrep
Author
unknown
Download Count*
LGPL Commons Clause
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

Semgrep CI docs

Defintion

Edit RuleView Source
Open in Playground 
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]
      vulnerability_class:
        - Code Injection
    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)')