r2dec-js icon indicating copy to clipboard operation
r2dec-js copied to clipboard

[WASM] Remove wasm from r2dec and make it as a standalone plugin

Open wargio opened this issue 6 years ago • 6 comments

Since WASM is particular, it would be better if we remove it from r2dec and make it as a standalone plugin.

WASM is like ESIL, a stack based language which requires some magic analysis which r2dec cannot do.

Current issues:

  • WASM flows guards are complex without emulation
  • Requires many interactions with r2 itself to get the call names/symbols

if anybody wants to help https://github.com/wargio/r2dec-wasm

wargio avatar Nov 30 '18 10:11 wargio

On 30 Nov 2018, at 11:12, Giovanni [email protected] wrote:

Since WASM is particular, it would be better if we remove it from r2dec and make it as a standalone plugin.

R2dec plugin?

WASM is like ESIL, a stack based language which requires some magic analysis which r2dec cannot do.

Current issues:

WASM flows guards are complex without emulation

Whats that

Requires many interactions with r2 itself to get the call names/symbols

Why?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

radare avatar Nov 30 '18 11:11 radare

because wasm is not easy, so i prefer to have a better decompilation via a separate r2 plugin

wargio avatar Nov 30 '18 11:11 wargio

R2 plugin why? Doesnt makes more sense to put yhis in r2dec?

On 30 Nov 2018, at 12:10, Giovanni [email protected] wrote:

because wasm is not easy, so i prefer to have a better decompilation via a separate r2 plugin

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

radare avatar Nov 30 '18 11:11 radare

no. the code structure i want to use is different

wargio avatar Nov 30 '18 12:11 wargio

Why not just keep this abstraction in r2dec instead of forking it?

On 30 Nov 2018, at 13:24, Giovanni [email protected] wrote:

no. the code structure i want to use is different

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

radare avatar Nov 30 '18 13:11 radare

because i don't know where to place it. it requires a different cycle of analysis. i'll think about it..

wargio avatar Dec 04 '18 17:12 wargio