LedgerHQ · cedelavergne-ledger · Feb 23, 2024 · Feb 23, 2024 · Feb 27, 2024 · Feb 27, 2024
diff --git a/.gitignore b/.gitignore
@@ -19,4 +19,5 @@ tests/elfs/*
 tests/snapshots-tmp
 
 .vscode
-.idea
+.idea
+venv
diff --git a/Makefile b/Makefile
@@ -268,6 +268,7 @@ endif
 
 # Allow usage of function from lib_standard_app/crypto_helpers.c
 APP_SOURCE_FILES += ${BOLOS_SDK}/lib_standard_app/crypto_helpers.c
+INCLUDES_PATH += ${BOLOS_SDK}/lib_standard_app
 
 ### initialize plugin SDK submodule if needed, rebuild it, and warn if a difference is noticed
 ifeq ($(CHAIN),ethereum)

diff --git a/doc/ethapp.adoc b/doc/ethapp.adoc
@@ -960,6 +960,159 @@ _Output data_
 
 None
 
+### Favorite Accounts
+
+#### Description
+
+This command allows to synchronize an Address Book between Ledger Live and the device.
+This allows user-friendly and easier address verifications.
+Also, the entries from the Address Book can be displayed as a QR code for easier control or sharing.
+
+#### Coding
+
+'Command'
+
+[width="80%"]
+|==============================================================================================================================
+| *CLA* | *INS*  | *P1*                                                 | *P2*             | *Lc*      | *Le*
+|   E0  |   30   |  01 : delete all accounts                            |  00              |  00       | empty
+|   E0  |   30   |  02 : get account                                    |  account Nb      |  00       | empty
+|   E0  |   30   |  03 : rename an account                              |  old name size   |  variable | see below
+|   E0  |   30   |  04 : update an account with address                 |  00              |  00       | see below
+|   E0  |   30   |  05 : update multiple accounts with address          |  00 : next data
+
+                                                                           01 : last data  | variable  | see below
+
+|   E0  |   30   |  06 : update an account with derivation path         |  00              |  00       | see below
+|   E0  |   30   |  07 : update multiple accounts with derivation path  |  00 : next data
+
+                                                                           01 : last data  | variable  | see below
+|==============================================================================================================================
+
+Commands summary:
+
+  - `delete all accounts`: _Purge_ the Address Book, and remove all entries.
+  - `get account`: Retrieve a specific Address Book entry; the Account Number is given in parameter.
+  - `rename an account`: _Change_ the Name assigned to an Account.
+  - `update an account`: _Add_ a new account, or _Rename_ an existing one.
+  - `update multiple accounts`: _Add_ new accounts and/or _Rename_ existing ones.
+This is a _chaining_ command, meaning several APDU must be sent successively,
+and the last one should have P2 = 0x01 to trig the confirmation screens.
+
+> For _update account_ commands, we have each time 2 variants:
+> 
+>  - Providing the full address
+>  - Providing the full derivation path, allowing the device to securely derivate and compute the address.
+
+_Input data_
+
+##### If P1 == `rename an account`
+
+[width="80%"]
+|==============================================================================================================================
+| *Description*                                                               | *Length*
+| Old Account Name (up to 16 ascii characters, 32 hex values - without '0x')  | variable (max 32)
+| New Account Name (up to 16 ascii characters, 32 hex values - without '0x')  | variable (max 32)
+|==============================================================================================================================
+
+> The _Old Account name_ length must be written in P2.
+
+##### If P1 == `update an account`
+
+This command is used to _Add_ a new Address Book entry. If the pair (*_Address_*, *_Chain Id_*) already exists,
+the command will be considered as a _Rename_.
+
+###### Case with Address
+
+[width="80%"]
+|==============================================================================================================================
+| *Description*                                                   | *Length*
+| Chain Id (8 Bytes)                                              | 16
+| Address (20 hex values without '0x')                            | 40
+| Name (up to 16 ascii characters, 32 hex values - without '0x')  | variable (max 32)
+|==============================================================================================================================
+
+###### Case with Derivation Path
+
+[width="80%"]
+|==============================================================================================================================
+| *Description*                                                   | *Length*
+| Chain Id (8 Bytes)                                              | 16
+| Derivation Path (1 Byte length + up to 5 x 4 Bytes)             | variable (max 42)
+| Name (up to 16 ascii characters, 32 hex values - without '0x')  | variable (max 32)
+|==============================================================================================================================
+
+
+##### If P1 == `update multiple account`
+
+Compared to the previous `update an account`, the principle here is to chain multiple APDU
+to update (_Add_/_Rename_) several Address Book entries at once.
+The review/confirmation will be displayed only once when all APDU have been sent.
+Thus, the last APDU must be distinguished using the P2 value.
+
+###### Case with Address
+
+[width="80%"]
+|==============================================================================================================================
+| *Description*                                                   | *Length*
+| Chain Id (8 Bytes)                                              | 16
+| Address (20 hex values without '0x')                            | 40
+| Name (up to 16 ascii characters, 32 hex values - without '0x')  | variable (max 32)
+|==============================================================================================================================
+
+###### Case with Derivation Path
+
+[width="80%"]
+|==============================================================================================================================
+| *Description*                                                   | *Length*
+| Chain Id (8 Bytes)                                              | 16
+| Derivation Path (1 Byte length + up to 5 x 4 Bytes)             | variable (max 42)
+| Name (up to 16 ascii characters, 32 hex values - without '0x')  | variable (max 32)
+|==============================================================================================================================
+
+_Output data_
+
+##### If P1 == `get account`
+
+[width="80%"]
+|==============================================================================================================================
+| *Description*                                                           | *Length*
+| Chain Id (8 Bytes)                                                      | 16
+| Address (20 hex values without '0x')                                    | 40
+| Account Name (up to 16 ascii characters, 32 hex values - without '0x')  | variable (max 32)
+|==============================================================================================================================
+
+
+_Status Word_
+
+The returned Status Word depends on the command:
+
+##### If P1 == `update multiple account`
+
+For each intermediate APDU, the Status Word will 0x61XX, giving the remaining available entries in the lower bits.
+For example, it will return `0x6102` when the command is correctly completed, and there are still `2` remaining (free) entries.
+Of course, after the *Last* one (with P2 = 0x01), it will be `0x9000`, indicating everything has been done and completed.
+
+##### Other commands
+
+The other commands will return `0x9000` when correctly completed.
+
+##### Errors
+
+In case of error, the _Status Word_ will be:
+
+[width="80%"]
+|==============================================================================================================================
+| *Description*                                                                     | *Status Word*
+| Error in P1 / P2 parameters                                                       | 0x6B00
+| Memory error: No more Address Book entries available                              | 0x6A80
+| Requested Data not found                                                          | 0x6A88
+| Add an account, but using an already assigned name                                | 0x6A80
+| Update an account, but APDU length is inconsistent                                | 0x6983
+| Add an account, but it is already present in the  Address Book                    | 0x6985
+|==============================================================================================================================
+
+
 
 ## Transport protocol
 

diff --git a/glyphs/filter32px.bmp b/glyphs/filter32px.bmp
diff --git a/glyphs/list32px.bmp b/glyphs/list32px.bmp
diff --git a/glyphs/plus32px.bmp b/glyphs/plus32px.bmp
diff --git a/glyphs/stax_app_chain_1.gif b/glyphs/stax_app_chain_1.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_1.gif
diff --git a/glyphs/stax_app_chain_1030.gif b/glyphs/stax_app_chain_1030.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_1030.gif
diff --git a/glyphs/stax_app_chain_108.gif b/glyphs/stax_app_chain_108.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_108.gif
diff --git a/glyphs/stax_app_chain_1284.gif b/glyphs/stax_app_chain_1284.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_1284.gif
diff --git a/glyphs/stax_app_chain_1285.gif b/glyphs/stax_app_chain_1285.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_1285.gif
diff --git a/glyphs/stax_app_chain_1313114.gif b/glyphs/stax_app_chain_1313114.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_1313114.gif
diff --git a/glyphs/stax_app_chain_137.gif b/glyphs/stax_app_chain_137.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_137.gif
diff --git a/glyphs/stax_app_chain_14.gif b/glyphs/stax_app_chain_14.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_14.gif
diff --git a/glyphs/stax_app_chain_16.gif b/glyphs/stax_app_chain_16.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_16.gif
diff --git a/glyphs/stax_app_chain_1620.gif b/glyphs/stax_app_chain_1620.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_1620.gif
diff --git a/glyphs/stax_app_chain_1818.gif b/glyphs/stax_app_chain_1818.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_1818.gif
diff --git a/glyphs/stax_app_chain_19.gif b/glyphs/stax_app_chain_19.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_19.gif
diff --git a/glyphs/stax_app_chain_1987.gif b/glyphs/stax_app_chain_1987.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_1987.gif
diff --git a/glyphs/stax_app_chain_199.gif b/glyphs/stax_app_chain_199.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_199.gif
diff --git a/glyphs/stax_app_chain_2.gif b/glyphs/stax_app_chain_2.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_2.gif
diff --git a/glyphs/stax_app_chain_200625.gif b/glyphs/stax_app_chain_200625.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_200625.gif
diff --git a/glyphs/stax_app_chain_20531811.gif b/glyphs/stax_app_chain_20531811.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_20531811.gif
diff --git a/glyphs/stax_app_chain_20531812.gif b/glyphs/stax_app_chain_20531812.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_20531812.gif
diff --git a/glyphs/stax_app_chain_237.gif b/glyphs/stax_app_chain_237.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_237.gif
diff --git a/glyphs/stax_app_chain_24.gif b/glyphs/stax_app_chain_24.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_24.gif
diff --git a/glyphs/stax_app_chain_24484.gif b/glyphs/stax_app_chain_24484.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_24484.gif
diff --git a/glyphs/stax_app_chain_246.gif b/glyphs/stax_app_chain_246.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_246.gif
diff --git a/glyphs/stax_app_chain_246529.gif b/glyphs/stax_app_chain_246529.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_246529.gif
diff --git a/glyphs/stax_app_chain_246785.gif b/glyphs/stax_app_chain_246785.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_246785.gif
diff --git a/glyphs/stax_app_chain_248.gif b/glyphs/stax_app_chain_248.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_248.gif
diff --git a/glyphs/stax_app_chain_269.gif b/glyphs/stax_app_chain_269.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_269.gif
diff --git a/glyphs/stax_app_chain_2894.gif b/glyphs/stax_app_chain_2894.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_2894.gif
diff --git a/glyphs/stax_app_chain_3.gif b/glyphs/stax_app_chain_3.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_3.gif
diff --git a/glyphs/stax_app_chain_30.gif b/glyphs/stax_app_chain_30.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_30.gif
diff --git a/glyphs/stax_app_chain_31.gif b/glyphs/stax_app_chain_31.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_31.gif
diff --git a/glyphs/stax_app_chain_31102.gif b/glyphs/stax_app_chain_31102.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_31102.gif
diff --git a/glyphs/stax_app_chain_3125659152.gif b/glyphs/stax_app_chain_3125659152.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_3125659152.gif
diff --git a/glyphs/stax_app_chain_336.gif b/glyphs/stax_app_chain_336.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_336.gif
diff --git a/glyphs/stax_app_chain_401697.gif b/glyphs/stax_app_chain_401697.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_401697.gif
diff --git a/glyphs/stax_app_chain_5.gif b/glyphs/stax_app_chain_5.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_5.gif
diff --git a/glyphs/stax_app_chain_50.gif b/glyphs/stax_app_chain_50.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_50.gif
diff --git a/glyphs/stax_app_chain_51.gif b/glyphs/stax_app_chain_51.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_51.gif
diff --git a/glyphs/stax_app_chain_56.gif b/glyphs/stax_app_chain_56.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_56.gif
diff --git a/glyphs/stax_app_chain_592.gif b/glyphs/stax_app_chain_592.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_592.gif
diff --git a/glyphs/stax_app_chain_60.gif b/glyphs/stax_app_chain_60.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_60.gif
diff --git a/glyphs/stax_app_chain_61.gif b/glyphs/stax_app_chain_61.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_61.gif
diff --git a/glyphs/stax_app_chain_62621.gif b/glyphs/stax_app_chain_62621.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_62621.gif
diff --git a/glyphs/stax_app_chain_64.gif b/glyphs/stax_app_chain_64.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_64.gif
diff --git a/glyphs/stax_app_chain_66.gif b/glyphs/stax_app_chain_66.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_66.gif
diff --git a/glyphs/stax_app_chain_7341.gif b/glyphs/stax_app_chain_7341.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_7341.gif
diff --git a/glyphs/stax_app_chain_73799.gif b/glyphs/stax_app_chain_73799.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_73799.gif
diff --git a/glyphs/stax_app_chain_76.gif b/glyphs/stax_app_chain_76.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_76.gif
diff --git a/glyphs/stax_app_chain_7762959.gif b/glyphs/stax_app_chain_7762959.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_7762959.gif
diff --git a/glyphs/stax_app_chain_78.gif b/glyphs/stax_app_chain_78.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_78.gif
diff --git a/glyphs/stax_app_chain_8.gif b/glyphs/stax_app_chain_8.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_8.gif
diff --git a/glyphs/stax_app_chain_82.gif b/glyphs/stax_app_chain_82.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_82.gif
diff --git a/glyphs/stax_app_chain_820.gif b/glyphs/stax_app_chain_820.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_820.gif
diff --git a/glyphs/stax_app_chain_846000.gif b/glyphs/stax_app_chain_846000.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_846000.gif
diff --git a/glyphs/stax_app_chain_88.gif b/glyphs/stax_app_chain_88.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_88.gif
diff --git a/glyphs/stax_app_chain_888.gif b/glyphs/stax_app_chain_888.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_888.gif
diff --git a/glyphs/stax_app_chain_99.gif b/glyphs/stax_app_chain_99.gif
@@ -0,0 +1 @@
+../icons/stax_app_chain_99.gif
diff --git a/glyphs/trash32px.bmp b/glyphs/trash32px.bmp
diff --git a/src/apdu_constants.h b/src/apdu_constants.h
@@ -24,6 +24,7 @@
 #define INS_EIP712_FILTERING                0x1E
 #define INS_ENS_GET_CHALLENGE               0x20
 #define INS_ENS_PROVIDE_INFO                0x22
+#define INS_FAVORITE_ACCOUNTS               0x30
 #define P1_CONFIRM                          0x01
 #define P1_NON_CONFIRM                      0x00
 #define P2_NO_CHAINCODE                     0x00
@@ -32,11 +33,19 @@
 #define P1_MORE                             0x80
 #define P2_EIP712_LEGACY_IMPLEM             0x00
 #define P2_EIP712_FULL_IMPLEM               0x01
+#define P1_FAVORITE_DELETE                  0x01
+#define P1_FAVORITE_GET                     0x02
+#define P1_FAVORITE_RENAME                  0x03
+#define P1_FAVORITE_UPDATE                  0x04
+#define P1_FAVORITE_UPDATE_MULTI            0x05
+#define P1_FAVORITE_UPDATE_PATH             0x06
+#define P1_FAVORITE_UPDATE_PATH_MULTI       0x07
 
 #define COMMON_CLA               0xB0
 #define COMMON_INS_GET_WALLET_ID 0x04
 
 #define APDU_RESPONSE_OK                      0x9000
+#define APDU_RESPONSE_AVAILABLE               0x6100
 #define APDU_RESPONSE_ERROR_NO_INFO           0x6a00
 #define APDU_RESPONSE_INVALID_DATA            0x6a80
 #define APDU_RESPONSE_INSUFFICIENT_MEMORY     0x6a84
@@ -112,6 +121,8 @@ void handlePerformPrivacyOperation(uint8_t p1,
                                    unsigned int *flags,
                                    unsigned int *tx);
 
+void getEthPublicKey(uint32_t *bip32Path, uint8_t bip32PathLength);
+
 #ifdef HAVE_ETH2
 
 void handleGetEth2PublicKey(uint8_t p1,