Fix HDPath validation and CLA in recovery app (#10)

* fix hdpath validation

* bump version

* fix cla for generic

* add API avail recovery app

* update zxlib and js

Author: abenso

Makefile

@@ -48,6 +48,14 @@ zemu_install: tests_tools_build
 
 test_all:
 	make zemu_install
+
+	echo "Testing recovery"
+	make clean
+	SUBSTRATE_PARSER_FULL=1 COIN=AVAIL_RECOVERY make
+	cd tests_zemu && yarn test_recovery && cd ..
+
+	echo "Testing standard"
+	make clean
 	SUBSTRATE_PARSER_FULL=1 make
 	make clean_build
 	SUBSTRATE_PARSER_FULL=1 SUPPORT_SR25519=1 make buildS

app/Makefile.version

@@ -3,4 +3,4 @@ APPVERSION_M=1
 # This is the `spec_version` field of `Runtime`
 APPVERSION_N=39
 # This is the patch version of this release
-APPVERSION_P=0
+APPVERSION_P=1

app/src/apdu_handler.c

@@ -43,8 +43,13 @@ void extractHDPath(uint32_t rx, uint32_t offset) {
 
     memcpy(hdPath, G_io_apdu_buffer + offset, sizeof(uint32_t) * HDPATH_LEN_DEFAULT);
 
+#ifdef RECOVERY_APP
+    const bool mainnet = hdPath[0] == HDPATH_0_DEFAULT &&
+                         hdPath[1] == HDPATH_1_RECOVERY;
+#else
     const bool mainnet = hdPath[0] == HDPATH_0_DEFAULT &&
                          hdPath[1] == HDPATH_1_DEFAULT;
+#endif
 
     if (!mainnet) {
         THROW(APDU_CODE_DATA_INVALID);
@@ -249,9 +254,15 @@ void handleApdu(volatile uint32_t *flags, volatile uint32_t *tx, uint32_t rx) {
     {
         TRY
         {
+#ifdef RECOVERY_APP
+            if (G_io_apdu_buffer[OFFSET_CLA] != CLA_RECOVERY) {
+                THROW(APDU_CODE_CLA_NOT_SUPPORTED);
+            }
+#else
             if (G_io_apdu_buffer[OFFSET_CLA] != CLA) {
                 THROW(APDU_CODE_CLA_NOT_SUPPORTED);
             }
+#endif
 
             if (rx < APDU_MIN_LENGTH) {
                 THROW(APDU_CODE_WRONG_LENGTH);

app/src/substrate/substrate_coin.h

@@ -21,13 +21,14 @@ extern "C" {
 
 #define COIN_ADDR_TYPE 42
 #define CLA 0xbc
+#define CLA_RECOVERY 0xbe
 
 #define INS_SIGN_RAW 0x03
 
 #define HDPATH_LEN_DEFAULT 5
 #define HDPATH_0_DEFAULT (0x80000000 | 0x2c)
 #define HDPATH_1_DEFAULT (0x80000000 | 0x2c5)
-#define HDPATH_1_RECOVERY (0x80000000 | 0x2c5)
+#define HDPATH_1_RECOVERY (0x80000000 | 0x162)
 
 #define HDPATH_2_DEFAULT (0x80000000u | 0u)
 #define HDPATH_3_DEFAULT (0u)

deps/ledger-zxlib

@@ -15,30 +15,31 @@
   ],
   "scripts": {
     "clean": "ts-node tests/pullImageKillOld.ts",
-    "test": "yarn clean && jest --maxConcurrency 3"
+    "test": "yarn clean && jest --maxConcurrency 3 --testPathIgnorePatterns recovery.test.ts",
+    "test_recovery": "yarn clean && jest --maxConcurrency 3 recovery.test.ts"
   },
   "dependencies": {
-    "@zondax/ledger-substrate": "^1.0.0",
-    "@zondax/zemu": "^0.51.0"
+    "@zondax/ledger-substrate": "^1.0.1",
+    "@zondax/zemu": "^0.53.0"
   },
   "devDependencies": {
     "@types/jest": "^29.5.14",
     "@types/ledgerhq__hw-transport": "^4.21.8",
-    "@typescript-eslint/eslint-plugin": "^8.12.2",
-    "@typescript-eslint/parser": "^8.12.2",
+    "@typescript-eslint/eslint-plugin": "^8.18.1",
+    "@typescript-eslint/parser": "^8.18.1",
     "blakejs": "^1.2.1",
     "crypto-js": "4.2.0",
     "ed25519-supercop": "^2.0.1",
-    "eslint": "^9.13.0",
+    "eslint": "^9.17.0",
     "eslint-config-prettier": "^9.1.0",
     "eslint-plugin-import": "^2.31.0",
-    "eslint-plugin-jest": "^28.8.3",
+    "eslint-plugin-jest": "^28.9.0",
     "eslint-plugin-prettier": "^5.2.1",
     "jest": "29.7.0",
     "jssha": "^3.3.1",
-    "prettier": "^3.3.3",
+    "prettier": "^3.4.2",
     "ts-jest": "^29.2.5",
     "ts-node": "^10.9.2",
-    "typescript": "^5.6.3"
+    "typescript": "^5.7.2"
   }
 }