airbus-seclab / cpu_rec

Recognize cpu instructions in an arbitrary binary file
Apache License 2.0
657 stars 60 forks source link

corpus: Add XtensaEB #17

Closed lorenz closed 1 year ago

lorenz commented 1 year ago

Xtensa cores can be configured both as little as well as big endian. This adds Xtensa's big endian variant to the corpus, using permissively-licensed code compiled from https://github.com/qca/open-ath9k-htc-firmware. Detection seems to be working properly, I ran it against a proprietary firmware (not in the corpus) which I know to be for a big endian Xtensa core and it picked it up.

I'm a bit unsure if I should rename the existing Xtensa corpus to XtensaLB, maybe (to match the format of other ISAs) they should also be changed to XTENSAel and XTENSAeb. This is just the minimal change to add the new corpus.

LRGH commented 1 year ago

Thanks. I merge your pull request now, and I might improve the naming in the future. But the naming is not that important...