Fix api_level extraction for binaries compiled with new SDK

LedgerHQ / cargo-ledger

(Moved in the ledger device rust SDK --> https://github.com/LedgerHQ/ledger-device-rust-sdk). Load code on a Ledger device with a `cargo` subcommand

Apache License 2.0

15 stars 12 forks source link

Fix api_level extraction for binaries compiled with new SDK #34

Closed siy closed 10 months ago

siy commented 10 months ago

The new SDK changed section names for meta information (API level, SDK version, etc.) and this results in incorrectly generated JSON for devices other than Nano S. The section content format also changed from binary to ASCII char. Fix provided in this PR enables processing of binaries compiled with new SDK.

xchapron-ledger commented 10 months ago

This fix is also included in this PR https://github.com/LedgerHQ/cargo-ledger/pull/31 We should merge one of the two soon.

xchapron-ledger commented 10 months ago

Fixed by https://github.com/LedgerHQ/cargo-ledger/pull/31