Disassembler not correctly representing negative offsets about rbpf HOT 6 CLOSED

lddw is also incorrectly represented. It only takes one imm, not two.

from rbpf.

I forgot rbpf has its own disassembler. Thanks for reporting these errors.

from rbpf.

lddw is also incorrectly represented. It only takes one imm, not two.

I don't see two immediate operands in disassembler output.

from rbpf.

I mean lddw is the only 16-byte instruction. It takes the imm from two consecutive 8-bytes (see https://github.com/solana-labs/rbpf/blob/main/src/ebpf.rs#L554). The disassembly only takes one imm.

from rbpf.

I mean lddw is the only 16-byte instruction. It takes the imm from two consecutive 8-bytes (see https://github.com/solana-labs/rbpf/blob/main/src/ebpf.rs#L554). The disassembly only takes one imm.

yes, it's a 64-bit immediate value, and it is decoded from the 128-bits of the instruction code. There are no two immediates. Not all bits of the 128-bit instruction code are used. Can you give an example of code that was disassembled incorrectly?

from rbpf.

Wait. Sorry, I'm stupid. I didn't realise that augment_lddw_unchecked had been applied prior to disassemble_instruction. I think it disassembles correctly.

from rbpf.