wargio / r2dec-js

radare2 plugin - converts asm to pseudo-C code.
514 stars 50 forks source link

Cleanup Optimization #157

Open Maijin opened 5 years ago

Maijin commented 5 years ago

In ioli0x00 ELF:

    0x0804841d mov eax, 0                            |     eax = 0;
    0x08048422 add eax, 0xf                          |     eax += 0xf;
    0x08048425 add eax, 0xf                          |     eax += 0xf;
    0x08048428 shr eax, 4                            |     eax >>= 4;
    0x0804842b shl eax, 4                            |     eax <<= 4;

Should be removed from the decompilation

radare commented 5 years ago

this requires code analysis, which is something r2dec cant do right now, so i dont expect this to be fixed yet. in radeco it should work fine

On 21 Apr 2019, at 18:35, Maijin notifications@github.com wrote:

In ioli0x00 ELF:

0x0804841d mov eax, 0                            |     eax = 0;
0x08048422 add eax, 0xf                          |     eax += 0xf;
0x08048425 add eax, 0xf                          |     eax += 0xf;
0x08048428 shr eax, 4                            |     eax >>= 4;
0x0804842b shl eax, 4                            |     eax <<= 4;

Should be removed from the decompilation

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/wargio/r2dec-js/issues/157, or mute the thread https://github.com/notifications/unsubscribe-auth/AAG75FQOQTCRL33U5YKYE2TPRSJTTANCNFSM4HHL7DWA.