Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

parse_expression API needs to use arch endianness #4404

Closed
amtal opened this issue Jun 13, 2023 · 1 comment
Closed

parse_expression API needs to use arch endianness #4404

amtal opened this issue Jun 13, 2023 · 1 comment
Labels
Component: Core Issue needs changes to the core Effort: Trivial Issue should take < 1 day Impact: Low Issue is a papercut or has a good, supported workaround Type: Bug Issue is a non-crashing bug with repro steps
Milestone
Coruscant

Comments

@amtal
Copy link

amtal commented Jun 13, 2023

Version and Platform (required):

  • Binary Ninja Version: 3.5.4320-dev
  • CPU Architecture: armv6

Bug Description: [addr] syntax does LE reads on BE arch

Steps To Reproduce: try dumping a command table from some BE net app fw

Expected Behavior: read 32bit words just like processor does, bv.arch.endianness is set correctly even in data region

Additional Information: please lift ARMV7_TEQ
@plafosse plafosse added Effort: Trivial Issue should take < 1 day Type: Bug Issue is a non-crashing bug with repro steps Impact: Low Issue is a papercut or has a good, supported workaround Component: Core Issue needs changes to the core labels Jun 13, 2023
@plafosse
Copy link
Member

This will be fixed in 3.5. 4329

@plafosse plafosse added this to the Coruscant milestone Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Component: Core Issue needs changes to the core Effort: Trivial Issue should take < 1 day Impact: Low Issue is a papercut or has a good, supported workaround Type: Bug Issue is a non-crashing bug with repro steps
Projects
None yet
Milestone
Coruscant
Development

No branches or pull requests
2 participants
@amtal @plafosse