Use API_LEVEL 26, and use Nano S+ headers instead of Nano X
What the developer wrote
Authored by Salvatore Ingala
Use API_LEVEL 26, and use Nano S+ headers instead of Nano X
What changed, and why it matters
This commit only changes the build configuration for unit tests. It updates the simulated Ledger device target from Nano X to Nano S+ and bumps the API level from 22 to 26. There is no change to the actual app code that runs on hardware wallets, so this does not affect user security.
No security action needed. Treat as routine build/test maintenance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies unit-tests/CMakeLists.txt, which is used for compiling unit tests against the Speculos emulator. It changes two preprocessor definitions: API_LEVEL from 22 to 26, and the SDK header path from nanox to nanos2. These are test-infrastructure-only changes that align the test environment with a newer Ledger SDK target. No application source code, cryptographic logic, or transaction handling is modified.
Changed components
unit-tests/CMakeLists.txtInspect captured patch +2 / −2
diff --git a/unit-tests/CMakeLists.txt b/unit-tests/CMakeLists.txt
index 107db93..6d1e477 100644
--- a/unit-tests/CMakeLists.txt
+++ b/unit-tests/CMakeLists.txt
@@ -213,7 +213,7 @@ if(SPECULOS AND SPECULOS_SRC)
target_compile_definitions(${target} PRIVATE
HAVE_HASH HAVE_RIPEMD160 HAVE_SHA256 HAVE_SHA512 HAVE_HMAC HAVE_MATH
HAVE_ECC HAVE_ECC_WEIERSTRASS HAVE_SECP256K1_CURVE HAVE_ECDSA HAVE_EDDSA
- API_LEVEL=22 OS_IO_SEPH_BUFFER_SIZE=272 IO_USB_MAX_ENDPOINTS=6
+ API_LEVEL=26 OS_IO_SEPH_BUFFER_SIZE=272 IO_USB_MAX_ENDPOINTS=6
BIP32_PUBKEY_VERSION=0x043587CF BIP44_COIN_TYPE=1
COIN_P2PKH_VERSION=111 COIN_P2SH_VERSION=196
COIN_COINID_SHORT=\"TEST\"
@@ -227,7 +227,7 @@ if(SPECULOS AND SPECULOS_SRC)
../src/handler/lib
../src/handler/sign_psbt
../src/musig
- $ENV{BOLOS_SDK}/target/nanox/include
+ $ENV{BOLOS_SDK}/target/nanos2/include
$ENV{BOLOS_SDK}/include
$ENV{BOLOS_SDK}/io/include
$ENV{BOLOS_SDK}/io_legacy/include
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.