Bugs in dvm.py

[GoogleCodeExporter]

Some bugs in dvm.py.

1. Unpack creates negative indexes in const-string(/jumbo) instructions. 

class Instruction21c(Instruction) :
    def __init__(self, cm, buff) :
...
      self.BBBB = unpack("=h", buff[2:4])[0]
                         ^^^^^
Must be 'H'.
The same is for Instruction31c

2. Instruction31c doesn't have method 'get_raw_string' which is used by 
decompiler. Something like:

    def get_raw_string(self) :
      return get_kind(self.cm, KIND_RAW_STRING, self.BBBBBBBB)

Original issue reported on code.google.com by neepo...@gmail.com on 13 Dec 2012 at 5:23

[GoogleCodeExporter]

Can you push a sample please ?

Original comment by anthony....@gmail.com on 13 Dec 2012 at 5:27

• Changed state: Accepted

[GoogleCodeExporter]

I think any big .dex will be fine. E.g. framework from emulator.

Original comment by neepo...@gmail.com on 13 Dec 2012 at 6:15

Attachments:

• classes.dex

[GoogleCodeExporter]

Could you try with my last commit ?

Thx

Original comment by anthony....@gmail.com on 13 Dec 2012 at 6:59

[GoogleCodeExporter]

4495    +         return pack("=HH", (self.AA << 8) | self.OP, self.BBBBBBBB)
                                          ^^^^
there is a mistake. must be =HI.
In other terms I think all is fine because I patched the same places.

P.S. Thank you for creating such nice reversing framework.

Original comment by neepo...@gmail.com on 13 Dec 2012 at 8:45

[GoogleCodeExporter]

Oops thx !

Fixed

Original comment by anthony....@gmail.com on 14 Dec 2012 at 7:24

• Changed state: Fixed