Skip to content

fix(deps): bump modelcontextprotocol/go-sdk to v1.4.1 (4 known CVEs) - #67

Merged
iFurySt merged 1 commit into
iFurySt:mainfrom
danyuchn:fix/go-sdk-cve-bump
Sep 9, 2026
Merged

fix(deps): bump modelcontextprotocol/go-sdk to v1.4.1 (4 known CVEs)#67
iFurySt merged 1 commit into
iFurySt:mainfrom
danyuchn:fix/go-sdk-cve-bump

Conversation

@danyuchn

@danyuchn danyuchn commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • scripts/computer-use-cli/go.mod pins github.com/modelcontextprotocol/go-sdk at v1.1.0, which carries four HIGH-severity CVEs fixed upstream:
  • Since computer-use-cli runs as a local MCP server, the DNS rebinding / CSRF pair means a malicious page open in the user's browser could reach the local server and invoke tools without authorization β€” a meaningfully dangerous combo for a tool whose whole job is to drive the screen.
  • Bumped to v1.4.1 (latest fixed release). go mod tidy also pulled in golang.org/x/sys, which had its own known CVE (CVE-2026-39824, integer overflow on Windows) β€” bumped that to v0.44.0 too.
  • No application code changed, only go.mod/go.sum.

Test plan

  • go build ./... β€” passes
  • go vet ./... β€” clean
  • go test ./... β€” all existing tests pass unchanged
  • trivy repo β€” 4 HIGH findings β†’ 0

πŸ€– Generated with Claude Code

The computer-use-cli module pinned go-sdk v1.1.0, which carries four
known HIGH-severity CVEs: DNS rebinding (CVE-2026-34742), CSRF-driven
remote tool execution (CVE-2026-33252), case-sensitivity handling
(CVE-2026-27896), and null-unicode JSON parsing (GHSA-q382-vc8q-7jhj).
Since this binary runs as a local MCP server, these are directly
exploitable by a malicious page open in the browser.

Also bumps the transitively pulled golang.org/x/sys to v0.44.0 to
clear an integer-overflow CVE (CVE-2026-39824) surfaced by go mod
tidy. Verified with `go build`, `go vet`, `go test ./...`, and a
trivy repo scan (4 HIGH -> 0 findings).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

@iFurySt iFurySt left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@iFurySt
iFurySt merged commit d9bcc74 into iFurySt:main Sep 9, 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.

2 participants