gemini-cli - 💡(How to fix) Fix Incomplete shell injection detection in ShellTool [1 comments, 2 participants]

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…
GitHub stats
google-gemini/gemini-cli#26788Fetched 2026-05-11 03:28:22
View on GitHub
Comments
1
Participants
2
Timeline
9
Reactions
0
Timeline (top)
labeled ×5added_to_project_v2 ×1commented ×1issue_type_added ×1
RAW_BUFFERClick to expand / collapse

What happened?

The detectCommandSubstitution check only targets specific subshell syntaxes and might miss other shell injection vectors like ;, &&, or || if the command is improperly concatenated elsewhere.

What did you expect to happen?

A more robust parsing-based approach should be used to validate the entire command structure before execution.

Client Information

<details> <summary>Client Information</summary>

OS: linux Date: Sunday, May 10, 2026 Gemini CLI: 0.42.0-nightly

</details>

Login information

Automated audit with Gemini CLI.

Anything else we need to know?

File: packages/core/src/tools/shell.ts, Line 493. Regex-based detection is often insufficient for complex shell languages.

Severity: High Area: area/security

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING