-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adds the Binary Ninja front-end plugin (MVP) + simplifies patch space 0 #240
Merged
Conversation
This file contains 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
Needs lots of work
This way we can easily save and load our patch configs inside of binja
This replaces the `overwrite` field of the config file. A patch size of zero implies that no code shall be overwritten. This way, we can disassemble as many instructions that would be overwritten by our trampoline and include those in the patch assembly.
This needs to be disambiguated from the `function` type, since VIBES represents them internally as the same thing
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR adds a front-end plugin for VIBES for the Binary Ninja reverse engineering suite. This serves as a significant step up in improving the user experience of VIBES. Key highlights:
.bndb
file).I've also simplified the way users can specify a patch that strictly inserts new code into the binary, denoted by a
patch-size
of0
. The corresponding changes tovibes-patch
reflect this.I should make it clear that this plugin is an MVP for getting a patch off the ground and sending the information off to the VIBES toolchain. In the future, we should be looking for user experience feedback to improve the usability of the plugin.