Security context

What an agent needs to avoid regressing past fixes and find the next vuln in this repo.

modelcontextprotocol/servers
main @ 562feeb
9
Fixes
0
CVEs
HIGH
Peak severity
100.0%
Coverage
Highlights
Path Traversal: 4 prior fixes. Scrutinize any change in this area.
src/git/src/mcp_server_git/server.py: most-fixed (5 issues). Treat as high-risk during review.
7 high-severity fixes in this history; regressions here are high-impact.
Recurring patterns

The bug types that recur here, drawn from past fixes, not open vulnerabilities.

Command Injection: Multiple git CLI sinks accept user-supplied arguments that can be interpreted as flags or options, leading to command injection. Fixes added both '-' rejection and '--' separator usage, indicating the guard must be applied consistently across all git operations.
Path Traversal: Path traversal in git operations (repo add, repo open) allowed staging or accessing files outside the configured repository. Fixes added path-boundary checks and symlink-resolved containment validation, but sibling operations may still lack these guards.
Argument Injection: Git diff and checkout accepted arguments starting with '-' leading to flag injection. The fix rejects such values and validates refs via rev_parse, but other git commands may not enforce the same rule.