samuelgursky/davinci-resolve-mcp vmain@123e844
This server scored F. Run it behind runtime policy enforcement so one bad tool call can't become an incident.
- criticalNo command-injection sinks — 11 occurrences
- mediumNo over-broad / destructive tools — 1 over-broad tool(s)
Security controls
Deterministic, evidence-backed. Score earned from passing controls; a failed guard caps it.
Security controls
Deterministic, evidence-backed. Score earned from passing controls; a failed guard caps it.
Model–MCP Runtime Guardrails
- FailUser-in-the-Loop / Approval Scopeguard1 over-broad tool(s)
No over-broad or destructive tools (arbitrary shell, bulk-delete) that warrant human approval. How many servers fail this?
offline_refFix: Scope tools narrowly; avoid arbitrary command execution and destructive defaults.
- PassIndirect Prompt Injection (IPI) Defensesguardno injection markers
Tool/prompt/resource text is free of hidden instructions that could hijack the agent.
- PassStrict JSON Schema EnforcementadditionalProperties:false present
Tool inputs are constrained (additionalProperties:false), so unexpected arguments can't be smuggled in.
- N/ATool Definition Integrityguardno prior version to diff
Every consecutive version pair is diffed for new injection markers or destructive scope — a risky diff anywhere in history is a rug-pull (fail, durable); benign description drift warns.
Application Security Checks
- FailNo command-injection sinksguard11 occurrences
Untrusted tool input reaching a shell yields remote code execution. How many servers fail this?
resolve-advanced/server/editorial.mjs:26resolve-advanced/server/editorial.mjs:63resolve-advanced/server/lut-apply.mjs:27resolve-advanced/server/media-inventory.mjs:20Fix: Never pass tool arguments to a shell; use argv arrays with shell disabled.
- WarnNo path traversalguard11 occurrences
Naive path checks let tools read/write outside intended directories (EscapeRoute-class).
bin/davinci-resolve-mcp.mjs:345resolve-advanced/scripts/calib-harness.mjs:24resolve-advanced/scripts/calib-harness.mjs:92resolve-advanced/server/editorial.mjs:115Fix: Resolve to a canonical path and verify containment; reject ../ and symlinks.
- WarnNo known-vulnerable dependencies9 of 192 runtime deps vulnerable; worst high
Runtime dependencies (parsed from the lockfile) are scanned against OSV.dev for published CVEs. Advisory: flagged dependencies lower the score but don't hard-cap it, since transitive reachability is unproven.
@hono/[email protected] — GHSA-frvp-7c67-39w9[email protected] — GHSA-xcpc-8h2w-3j85[email protected] — GHSA-4c8g-83qw-93j6[email protected] — GHSA-7p8r-x3mc-p8w7Fix: Upgrade the flagged dependencies past their fixed versions.
- PassNo dynamic code executionguardno sinks found
eval()/exec()/Function() on tool-derived strings allows arbitrary code execution.
- PassNo SSRF sinksguardno sinks found
Fetching tool-supplied URLs can pivot into internal networks and metadata services.
- PassNo unsafe deserializationguardno sinks found
pickle/yaml.load/etc. on untrusted data can execute code.
- PassNo committed secretsguardno secrets found
Hardcoded keys/tokens in published source are live credentials an attacker can use.
- PassCredentials sourced from environmentreads credentials from environment
Reading secrets from env/secret stores avoids hardcoding them.
- PassDependencies pinned (lockfile)lockfile present
A lockfile makes installs reproducible and resistant to silent dependency swaps.
- PassActively maintainedrecent commits
Unmaintained servers don't receive security fixes.
- PassRepository not archivedguardactive
Archived repositories will never be patched.
- PassDeclares a licenseMIT
A clear license is required for legal enterprise use.
- PassHas a security policySECURITY.md present
A SECURITY.md gives a private path to report vulnerabilities.
- PassAdoption & popularityestablished adoption
A small, capped nudge from stars/downloads — widely-used servers get more eyes on bugs. It can never offset a real security failure.
- Not checkedSigned releasesnot evaluated
Signed releases let consumers verify artifacts weren't tampered with.
- N/ANo install/post-install scriptsguardno published package
install hooks run arbitrary code on every consumer at install time.
- N/APackage name not typosquattingguardno published package
Names mimicking popular packages are a common malware delivery vector.
- N/APublished with provenanceno published package
Build provenance attests the artifact was built from the claimed source by CI.
- N/AEstablished maintainerno published package
Brand-new / single anonymous maintainers raise takeover and malware risk.
Transport & Trust Model
- WarnExecution Sandboxingruns natively (no container image)
A container/sandbox image limits blast radius; a server that runs natively has full host access.
Fix: Ship a Dockerfile/Containerfile (or document a sandboxed run) so the server runs isolated.
- PassNetwork Exposureno bind-all detected
Binding 0.0.0.0 or exposing debug inspectors widens the attack surface.
- N/ATransport Encryption (TLS)guardno remote endpoints
Plaintext HTTP exposes traffic and bearer tokens to interception.
- N/AIAM / Authentication Scopingno remote endpoints
OAuth 2.1 / Protected Resource Metadata gates who can invoke tools.
- N/ALive Endpoint Reachablenot dynamically scanned
A dynamic scan connected to the declared remote endpoint and it responded — verified live, not a dead URL.
- N/AAuthentication Enforced (live)not dynamically scanned
If the server declares auth is required, it must actually reject anonymous clients. Serving tools to unauthenticated callers is a real exposure.