Skip to content

Add advisory for rmcp DNS rebinding - #2998

Merged
djc merged 1 commit into
rustsec:mainfrom
DaleSeo:add-rmcp-advisory
Jun 29, 2026
Merged

Add advisory for rmcp DNS rebinding#2998
djc merged 1 commit into
rustsec:mainfrom
DaleSeo:add-rmcp-advisory

Conversation

@DaleSeo

@DaleSeo DaleSeo commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Affected crate(s)

  • rmcp (13,797,440 total downloads; 7,472,054 recent downloads per crates.io)

Links to upstream issue(s) or PR(s)

Severity

High. A malicious public website could use DNS rebinding to send requests to a locally running rmcp Streamable HTTP server, allowing it to enumerate and invoke exposed MCP tools and access resources or prompts available through that server. The upstream GHSA uses CVSS 3.1 score 8.8.

Checklist

  • Advisory filename(s) starts with RUSTSEC-0000-0000 as the ID
  • date field is set to the public disclosure date
  • Contains a concise and descriptive title after advisory metadata
  • Asked maintainer(s) if publishing an advisory is appropriate

Testing

  • Added the advisory under crates/rmcp/RUSTSEC-0000-0000.md using the RustSec template.
  • Ran rustsec-admin lint; it currently stops on an existing pnet advisory before reaching this new file (crates.io package name does not match package name in advisory for pnet in RUSTSEC-2019-0037).

@djc
djc merged commit 5e2f36f into rustsec:main Jun 29, 2026
1 check passed
@djc

djc commented Jun 29, 2026

Copy link
Copy Markdown
Member

Thanks! Going to treat the existence of a GitHub advisory as signal that a RustSec advisory is probably okay.

@djc

djc commented Jun 29, 2026

Copy link
Copy Markdown
Member
  • Ran rustsec-admin lint; it currently stops on an existing pnet advisory before reaching this new file (crates.io package name does not match package name in advisory for pnet in RUSTSEC-2019-0037).

FWIW, I cannot reproduce this. Do you have an older version of rustsec-admin installed?

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