Skip to content

Conversation

@ltlly
Copy link
Contributor

@ltlly ltlly commented Apr 10, 2025

Add debugger-related mpc function, including functionality to retrieve all registers, call stack, byte values, strings, and more.

…e all registers, call stack, byte values, strings, and more.
@mrexodia
Copy link
Owner

Thanks for your contribution! As-is I do not want to merge it for a few reasons:

  1. All the code was reformatted, which I do not want. PEP8 is wrong 😅
  2. Adding debugger support opens the MCP up to RCE exploits with prompt injection. If we want to support this it will need to be explicitly enabled when starting the MCP server, so that requires additional design work.
  3. Many functions unrelated to the debugger are also added, please split them into a separate PR.

@ltlly ltlly closed this Apr 16, 2025
@mrexodia
Copy link
Owner

Why did you close it? I'm happy to incorporate the changes myself.

@mrexodia mrexodia merged commit 66b22b0 into mrexodia:main Apr 18, 2025
@mrexodia
Copy link
Owner

Thanks!

@mrexodia mrexodia mentioned this pull request Apr 18, 2025
can1357 pushed a commit to can1357/ida-pro-mcp that referenced this pull request Dec 19, 2025
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