Skip to content

fix: semgrep-subprocess-shell-true#1

Merged
girichinna27 merged 1 commit into
scanning-1from
fix/semgrep-subprocess-shell-true-2694-XaudKbc6Fq
Feb 13, 2026
Merged

fix: semgrep-subprocess-shell-true#1
girichinna27 merged 1 commit into
scanning-1from
fix/semgrep-subprocess-shell-true-2694-XaudKbc6Fq

Conversation

@ai-remediation
Copy link
Copy Markdown

Pull Request — Semgrep Rule Fix

  • Rule ID: subprocess-shell-true
  • Rule Message: Found 'subprocess' function 'Popen' with 'shell=True'. This is dangerous because this call will spawn the command using a shell process. Doing so propagates current shell settings and variables, which makes it much easier for a malicious actor to execute commands. Use 'shell=False' instead.
  • File Path: unsloth/chat_templates.py
  • Line: 2694

@girichinna27 girichinna27 merged commit d7db609 into scanning-1 Feb 13, 2026
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