Skip to content

Comments

Log an error when endianness change is not compatible with arch#5945

Open
wargio wants to merge 4 commits intodevfrom
notify-when-endian-not-compatible
Open

Log an error when endianness change is not compatible with arch#5945
wargio wants to merge 4 commits intodevfrom
notify-when-endian-not-compatible

Conversation

@wargio
Copy link
Member

@wargio wargio commented Feb 19, 2026

Your checklist for this pull request

  • I've read the guidelines for contributing to this repository.
  • I made sure to follow the project's coding style.
  • I've documented every RZ_API function and struct this PR changes.
  • I've added tests that prove my changes are effective (required for changes to RZ_API).
  • I've updated the Rizin book with the relevant information (if needed).
  • I've used AI tools to generate fully or partially these code changes and I'm sure the changes are not copyrighted by somebody else.

Detailed description

This generates a log when the switch to a specific endianness fails.

[0x00000000]> e asm.arch=x86
[0x00000000]> e cfg.bigendian
false
[0x00000000]> e cfg.bigendian=true
ERROR: core: cannot change to big endian (arch 'x86' supports only little endian)
[0x00000000]> e asm.arch=sysz 
[0x00000000]> e cfg.bigendian
true
[0x00000000]> e cfg.bigendian=false
ERROR: core: cannot change to little endian (arch 'sysz' supports only big endian)

notxvilka

This comment was marked as resolved.

@wargio wargio force-pushed the notify-when-endian-not-compatible branch from 1030402 to 189b60b Compare February 20, 2026 03:58
notxvilka

This comment was marked as resolved.

@wargio
Copy link
Member Author

wargio commented Feb 20, 2026

The issue is when setting the default endianness, i forgot to change that :D

@github-actions github-actions bot added the RzBin label Feb 20, 2026
@wargio wargio force-pushed the notify-when-endian-not-compatible branch from 754bc8c to c86c0b8 Compare February 20, 2026 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants