Skip to content

Feature/angr decompilation view#436

Merged
rbs-jacob merged 24 commits into
redballoonsecurity:masterfrom
dannyp303:feature/angr_decompilation_view
Mar 4, 2024
Merged

Feature/angr decompilation view#436
rbs-jacob merged 24 commits into
redballoonsecurity:masterfrom
dannyp303:feature/angr_decompilation_view

Conversation

@dannyp303
Copy link
Copy Markdown
Contributor

One sentence summary of this PR (This should go in the CHANGELOG!)
Add a content view and decompilation view to the OFRAK gui.
Link to Related Issue(s)

Please describe the changes in your request.

Anyone you think should look at this, specifically?

@dannyp303 dannyp303 requested a review from rbs-jacob February 29, 2024 20:59
Copy link
Copy Markdown
Member

@rbs-jacob rbs-jacob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some suggested changes.

One thing I'm confused about is: where is the code to make angr pull the decompilation output into OFRAK? I don't see that in this PR. Is that already a part of OFRAK somewhere?

Comment thread ofrak_core/CHANGELOG.md Outdated
Comment thread frontend/src/hex/MinimapView.svelte
Comment thread frontend/src/views/DecompilationView.svelte Outdated
Comment thread frontend/src/views/DecompilationView.svelte Outdated
Comment thread frontend/src/views/ContentView.svelte Outdated
Comment thread frontend/src/utils/Tabs.svelte Outdated
Comment thread frontend/src/utils/Tabs.svelte Outdated
Comment thread frontend/src/utils/Tabs.svelte
Comment thread frontend/src/views/DecompilationView.svelte
Comment thread frontend/src/views/ContentView.svelte Outdated
dannyp303 and others added 6 commits March 1, 2024 15:28
Co-authored-by: Jacob Strieb <99368685+rbs-jacob@users.noreply.github.com>
Co-authored-by: Jacob Strieb <99368685+rbs-jacob@users.noreply.github.com>
Co-authored-by: Jacob Strieb <99368685+rbs-jacob@users.noreply.github.com>
@dannyp303 dannyp303 requested a review from rbs-jacob March 1, 2024 21:43
Copy link
Copy Markdown
Member

@rbs-jacob rbs-jacob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Last small changes. But it looks pretty much good to go to me!

Comment thread disassemblers/ofrak_angr/ofrak_angr/components/angr_decompilation_analyzer.py Outdated
Comment thread frontend/src/utils/Tabs.svelte Outdated
Comment thread frontend/src/utils/Tabs.svelte Outdated
Comment thread frontend/src/views/ContentView.svelte
@dannyp303 dannyp303 requested a review from rbs-jacob March 1, 2024 23:05
Copy link
Copy Markdown
Member

@rbs-jacob rbs-jacob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One last thing then good to go

Comment thread frontend/src/utils/Tabs.svelte Outdated
Comment thread .gitattributes Outdated
@rbs-jacob rbs-jacob merged commit 62ba303 into redballoonsecurity:master Mar 4, 2024
ANogin pushed a commit to ANogin/ofrak that referenced this pull request May 30, 2024

---------

Co-authored-by: Dan Pesce <dan@redballoonsecurity.com>
Co-authored-by: Jacob Strieb <99368685+rbs-jacob@users.noreply.github.com>
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