search

tkmru / nao

Simple No-meaning Assembly Omitter for IDA Pro (This is just a prototype)
GNU General Public License v3.0
275 stars 49 forks source link

Error : String index out of range #27

Closed Spl3en closed 7 years ago

Spl3en commented 7 years ago

Hello,

I tried running this plugin on a CTF binary from GreHack 2015 (This one) and the plugin crashes when I try to run dead code elimination.

Here is the python traceback : 

Traceback (most recent call last):
  File "D:/Logiciels/IDA6.8/scripts/nao/nao.py", line 201, in create_view
    view.Create()
  File "D:/Logiciels/IDA6.8/scripts/nao/nao.py", line 104, in Create
    self.load(ea)
  File "D:/Logiciels/IDA6.8/scripts/nao/nao.py", line 118, in load
    if not self.eliminate_deadcode(ea):
  File "D:/Logiciels/IDA6.8/scripts/nao/nao.py", line 149, in eliminate_deadcode
    if b'\x90' != i[1][0]:  # check dead code
IndexError: string index out of range
tkmru commented 7 years ago

Thanks. I will try this later.

tkmru commented 7 years ago

@Spl3en I fixed it, Thanks for bug report.