Warn when TTD is used without the WinDbg/TTD package (Fixes #1103) - #1143
Open
xusheng6 wants to merge 1 commit into
Open
Warn when TTD is used without the WinDbg/TTD package (Fixes #1103)#1143xusheng6 wants to merge 1 commit into
xusheng6 wants to merge 1 commit into
Conversation
The debugger only loads the WinDbg/TTD copy that Binary Ninja downloads itself, but a self-installed WinDbg looks like it should work, so the mistake surfaced as an opaque "Failed to initialize DbgEng". Check for the replay engine and the recorder before launching a DBGENG_TTD session and before the TTD record/attach dialogs, and explain what is missing with a button to install it. The core-side errors carry the same explanation. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #1103.
The user can try to start a TTD debugging session without installing WinDbg/TTD first. A common misconception is that they install WinDbg separately and expect that to work with Binary Ninja. However, due to Windows App restrictions, this would not work
This PR checks if the TTD DLLs can be found, and if not, let them know the debugging will fail, and prompt them to install it. It also specifically points out that a previously installed WinDbg does not work and the installation must go through binja