-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Support all analysis info required for Diaphora and create a script to extract #2742
Comments
Oh! And some structure or whatever representing the control flow graph of each function. |
Can you specify the type of each field? array of strings, graph, etc? Except the the decompilation, r2 can provide the all this information. and the decompilation can be extracted by using the retdec plugin or pdc/radeco in the future if needed. Also, mark which of those fields are optional |
Waiting for @joxeankoret to provide an interface class to fill it with r2pipe magic |
Any updates about this? |
I guess we should do everything. But its ok for 1.0
|
@sushant94 @oddcoder would be useful also for 'radare2 <-> radeco' interaction. |
see |
the |
One of the problems I'm facing: I have no idea how to, simply, get the number of edges in a function. That's it, for a function graph |
https://twitter.com/trufae/status/793504061152690178
|
When you say V and E do you mean for the BBS in a function, or its in and On Nov 1, 2016 10:49 AM, "radare" notifications@github.com wrote:
|
With |
All those requirements are fullfilled, what is remaining for this to happen @joxeankoret |
GUI support, mainly. Even supposing that I can port the internal export logic, there is no way I can display the results anywhere unless I create myself a new independent GUI. |
Buy a mac
… On 12 Mar 2017, at 12:03, Joxean ***@***.***> wrote:
GUI support, mainly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
No srsly. Cant you just make a command line interface? Visual mode should be enough for most of that or at least a batch mode
… On 12 Mar 2017, at 12:03, Joxean ***@***.***> wrote:
GUI support, mainly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
The Visual mode is cool, but non practical for diffing, I guess. Command line diffing? For example, I had a project where many functions had hundreds and even thousands of basic blocks. How am I supposed to analyse it with a command line version? No, you need a graph view with an overview or something similar. That said, one Diaphora's side project is to create an independent GUI tool. |
is you create proper primitives the gui will be a separate part totally independent from the diffing engine and the backend source. so it doesnt matters if its a gui or a cli or a cui.
i understand a proper gui is needed for managing this kind of things for big projects, but that’s not a reason for blocking access to this, mainly because you cant test graphical apps.. well you can but its silly and having stuff done without tests is dangerous
… On 13 Mar 2017, at 17:36, Joxean ***@***.***> wrote:
The Visual mode is cool, but non practical for diffing, I guess. Command line diffing? For example, I had a project where many functions had hundreds and even thousands of basic blocks. How am I supposed to analyse it with a command line version? No, you need a graph view with an overview or something similar.
That said, one Diaphora's side project is to create an independent GUI tool.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#2742 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA3-lmIjw0YpKWLZd84WM0pnQDqhtlb4ks5rlXB-gaJpZM4E_eoC>.
|
|
Writing an exporter should be rather easy. I will start with this part. |
Can this be closed? Since it's completely done. |
@radare so what do you think? everything on radare2 side is ready/ |
Yes i think we can close this issue and open another one to write the script that exposes all that info instead. Tired of waiting, this stupid python script is taking more than 2 years to be implemented. |
What is required:
The text was updated successfully, but these errors were encountered: