Skip to content
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

Use LibBS to collect artifacts from decompilers #84

Open
mahaloz opened this issue Jan 4, 2024 · 2 comments
Open

Use LibBS to collect artifacts from decompilers #84

mahaloz opened this issue Jan 4, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@mahaloz
Copy link
Owner

mahaloz commented Jan 4, 2024

Overview

LibBS, an abstracted library for working with decompilers from the BinSync team, implements much of what we do in XMLRPC servers for the many decompilers. It also reimplements all the Java Ghidra code we have into Python.

This change would remove all code from the decompilers folder, since it is contained in LibBS

Benefits

  • Reduces the overlapping code I've written for BinSync and DAILA contained in d2d
  • We gain bug fixes from fixes to BinSync
  • We get one step closer to Add Support for Struct Imports #36, since we will now have struct exporting supported in all decompilers

Disadvantages

  • Debugging for new people to the project gets harder since now they need to clone LibBS to debug problems that occur inside the decompiler servers accurately (harder debugging)
  • Installing this tool gets a little harder, since now decomp2dbg, the Python package, will need to be installed in the same Python env as the decompiler. For Ghidra, this is trivial since one does not exist; they install it into the Python found in the path. For Binja, it's a possible pain. IDA is ok.

This PR is stalled by this issue from LibBS:
binsync/libbs#16

Once it is done, this can be worked on.
Since this is open-source, if the community is reading this issue now, I highly recommend you react to the issue with a 👍 or a 👎 depending on how you feel about the PR. I.e., if you want this, 👍. Otherwise, 👎.

@k4lizen
Copy link

k4lizen commented Oct 4, 2024

Seems the libbs PR was merged!

@mahaloz
Copy link
Owner Author

mahaloz commented Oct 4, 2024

Good point! Now I just need to find some time 😢. Struct support would be amazing, but caching would be EVEN BETTER.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants