BPF testcases fail due to a bug in PyVEX about angr-platforms HOT 7 CLOSED

I actually successfully ran the tests before merging. Hmm...
Ninja-edit: I just ran fish's test again and it works locally. Looks like there's an incongruency w/ CI somehow.

from angr-platforms.

I figured out what is going on. Here is the chat log copied from the angr core channel:

fish [1:50 AM]
@lockshaw @subwire
in pyvex/lift/__init__.py:lift()

    for lifter in lifters:
            ...
            next_irsb_part = lifter(arch, addr)._lift(u_data, bytes_offset, max_bytes, max_inst, opt_level, traceflags, allow_lookback)

this is not ideal, since sometimes we have more than one lifters registered (apart from the default VEX lifter), and it's possible that the both lifters are able to lift that instruction

[1:51]
this is exactly why test cases on travis CI is failing: both BPF and MSP430 lifters are currently registered in the CI, and the MSP430 lifter is called first and tries to lift a BPF instruction

[1:55]
here is the old way:

    for lifter, registered_arch in lifters:
        if not isinstance(arch, registered_arch):
            continue

I think the old way is better - we don't accidentally use lifters for one architecture to lift blocks in another architecture

[1:56]
Here is my proposal: we keep a dict that maps each supported architecture to a list of lifters, which will allow us to do the following:

for lifter in arch_lifters[arch.name]:
   ....

what do you think?

EDIT: Fix format issues.

from angr-platforms.

The related PyVEX commit: angr/pyvex@e47ca1b5

from angr-platforms.

To reproduce the issue locally: Add the following line at the beginning of test_bpf_idea.py, right after import angr:

from angr_platforms.msp430 import *

from angr-platforms.

Now this explains everything.
Yeah, let's add the arch, that makes perfect sense.

from angr-platforms.

This issue has been marked as stale because it has no recent activity. Please comment or add the pinned tag to prevent this issue from being closed.

from angr-platforms.

This issue has been closed due to inactivity.

from angr-platforms.