Merge pull request #6898

593ae2e device: Ledger - update status codes (xiphon)

2 files changed, 43 insertions, 65 deletions

diff --git a/src/device/device_ledger.cpp b/src/device/device_ledger.cpp
index 3e0afeb65..384b776ee 100644
--- a/src/device/device_ledger.cpp
+++ b/src/device/device_ledger.cpp
@@ -83,44 +83,33 @@ namespace hw {
     // Must be sorted in ascending order by the code
     #define LEDGER_STATUS(status) {status, #status}
     constexpr Status status_codes[] = {
-      LEDGER_STATUS(SW_BYTES_REMAINING_00),
-      LEDGER_STATUS(SW_WARNING_STATE_UNCHANGED),
-      LEDGER_STATUS(SW_STATE_TERMINATED),
-      LEDGER_STATUS(SW_MORE_DATA_AVAILABLE),
+      LEDGER_STATUS(SW_OK),
       LEDGER_STATUS(SW_WRONG_LENGTH),
-      LEDGER_STATUS(SW_LOGICAL_CHANNEL_NOT_SUPPORTED),
-      LEDGER_STATUS(SW_SECURE_MESSAGING_NOT_SUPPORTED),
-      LEDGER_STATUS(SW_LAST_COMMAND_EXPECTED),
-      LEDGER_STATUS(SW_COMMAND_CHAINING_NOT_SUPPORTED),
+      LEDGER_STATUS(SW_SECURITY_PIN_LOCKED),
       LEDGER_STATUS(SW_SECURITY_LOAD_KEY),
       LEDGER_STATUS(SW_SECURITY_COMMITMENT_CONTROL),
       LEDGER_STATUS(SW_SECURITY_AMOUNT_CHAIN_CONTROL),
       LEDGER_STATUS(SW_SECURITY_COMMITMENT_CHAIN_CONTROL),
       LEDGER_STATUS(SW_SECURITY_OUTKEYS_CHAIN_CONTROL),
       LEDGER_STATUS(SW_SECURITY_MAXOUTPUT_REACHED),
-      LEDGER_STATUS(SW_SECURITY_TRUSTED_INPUT),
-      LEDGER_STATUS(SW_CLIENT_NOT_SUPPORTED),
-      LEDGER_STATUS(SW_SECURITY_STATUS_NOT_SATISFIED),
-      LEDGER_STATUS(SW_FILE_INVALID),
-      LEDGER_STATUS(SW_PIN_BLOCKED),
-      LEDGER_STATUS(SW_DATA_INVALID),
-      LEDGER_STATUS(SW_CONDITIONS_NOT_SATISFIED),
+      LEDGER_STATUS(SW_SECURITY_HMAC),
+      LEDGER_STATUS(SW_SECURITY_RANGE_VALUE),
+      LEDGER_STATUS(SW_SECURITY_INTERNAL),
+      LEDGER_STATUS(SW_SECURITY_MAX_SIGNATURE_REACHED),
+      LEDGER_STATUS(SW_SECURITY_PREFIX_HASH),
+      LEDGER_STATUS(SW_SECURITY_LOCKED),
       LEDGER_STATUS(SW_COMMAND_NOT_ALLOWED),
-      LEDGER_STATUS(SW_APPLET_SELECT_FAILED),
+      LEDGER_STATUS(SW_SUBCOMMAND_NOT_ALLOWED),
+      LEDGER_STATUS(SW_DENY),
+      LEDGER_STATUS(SW_KEY_NOT_SET),
       LEDGER_STATUS(SW_WRONG_DATA),
-      LEDGER_STATUS(SW_FUNC_NOT_SUPPORTED),
-      LEDGER_STATUS(SW_FILE_NOT_FOUND),
-      LEDGER_STATUS(SW_RECORD_NOT_FOUND),
-      LEDGER_STATUS(SW_FILE_FULL),
-      LEDGER_STATUS(SW_INCORRECT_P1P2),
-      LEDGER_STATUS(SW_REFERENCED_DATA_NOT_FOUND),
+      LEDGER_STATUS(SW_WRONG_DATA_RANGE),
+      LEDGER_STATUS(SW_IO_FULL),
+      LEDGER_STATUS(SW_CLIENT_NOT_SUPPORTED),
       LEDGER_STATUS(SW_WRONG_P1P2),
-      LEDGER_STATUS(SW_CORRECT_LENGTH_00),
       LEDGER_STATUS(SW_INS_NOT_SUPPORTED),
-      LEDGER_STATUS(SW_CLA_NOT_SUPPORTED),
-      LEDGER_STATUS(SW_UNKNOWN),
-      LEDGER_STATUS(SW_OK),
-      LEDGER_STATUS(SW_ALGORITHM_UNSUPPORTED)
+      LEDGER_STATUS(SW_PROTOCOL_NOT_SUPPORTED),
+      LEDGER_STATUS(SW_UNKNOWN)
     };
 
     const char *Status::to_string(unsigned int code)
diff --git a/src/device/device_ledger.hpp b/src/device/device_ledger.hpp
index 5cb834e02..00ff05ec5 100644
--- a/src/device/device_ledger.hpp
+++ b/src/device/device_ledger.hpp
@@ -59,44 +59,33 @@ namespace hw {
     #ifdef WITH_DEVICE_LEDGER
 
     // Origin: https://github.com/LedgerHQ/ledger-app-monero/blob/master/src/monero_types.h
-    #define SW_BYTES_REMAINING_00                0x6100
-    #define SW_WARNING_STATE_UNCHANGED           0x6200
-    #define SW_STATE_TERMINATED                  0x6285
-    #define SW_MORE_DATA_AVAILABLE               0x6310
-    #define SW_WRONG_LENGTH                      0x6700
-    #define SW_LOGICAL_CHANNEL_NOT_SUPPORTED     0x6881
-    #define SW_SECURE_MESSAGING_NOT_SUPPORTED    0x6882
-    #define SW_LAST_COMMAND_EXPECTED             0x6883
-    #define SW_COMMAND_CHAINING_NOT_SUPPORTED    0x6884
-    #define SW_SECURITY_LOAD_KEY                 0x6900
-    #define SW_SECURITY_COMMITMENT_CONTROL       0x6911
-    #define SW_SECURITY_AMOUNT_CHAIN_CONTROL     0x6912
-    #define SW_SECURITY_COMMITMENT_CHAIN_CONTROL 0x6913
-    #define SW_SECURITY_OUTKEYS_CHAIN_CONTROL    0x6914
-    #define SW_SECURITY_MAXOUTPUT_REACHED        0x6915
-    #define SW_SECURITY_TRUSTED_INPUT            0x6916
-    #define SW_CLIENT_NOT_SUPPORTED              0x6930
-    #define SW_SECURITY_STATUS_NOT_SATISFIED     0x6982
-    #define SW_FILE_INVALID                      0x6983
-    #define SW_PIN_BLOCKED                       0x6983
-    #define SW_DATA_INVALID                      0x6984
-    #define SW_CONDITIONS_NOT_SATISFIED          0x6985
-    #define SW_COMMAND_NOT_ALLOWED               0x6986
-    #define SW_APPLET_SELECT_FAILED              0x6999
-    #define SW_WRONG_DATA                        0x6a80
-    #define SW_FUNC_NOT_SUPPORTED                0x6a81
-    #define SW_FILE_NOT_FOUND                    0x6a82
-    #define SW_RECORD_NOT_FOUND                  0x6a83
-    #define SW_FILE_FULL                         0x6a84
-    #define SW_INCORRECT_P1P2                    0x6a86
-    #define SW_REFERENCED_DATA_NOT_FOUND         0x6a88
-    #define SW_WRONG_P1P2                        0x6b00
-    #define SW_CORRECT_LENGTH_00                 0x6c00
-    #define SW_INS_NOT_SUPPORTED                 0x6d00
-    #define SW_CLA_NOT_SUPPORTED                 0x6e00
-    #define SW_UNKNOWN                           0x6f00
-    #define SW_OK                                0x9000
-    #define SW_ALGORITHM_UNSUPPORTED             0x9484
+    #define SW_OK                                   0x9000
+    #define SW_WRONG_LENGTH                         0x6700
+    #define SW_SECURITY_PIN_LOCKED                  0x6910
+    #define SW_SECURITY_LOAD_KEY                    0x6911
+    #define SW_SECURITY_COMMITMENT_CONTROL          0x6912
+    #define SW_SECURITY_AMOUNT_CHAIN_CONTROL        0x6913
+    #define SW_SECURITY_COMMITMENT_CHAIN_CONTROL    0x6914
+    #define SW_SECURITY_OUTKEYS_CHAIN_CONTROL       0x6915
+    #define SW_SECURITY_MAXOUTPUT_REACHED           0x6916
+    #define SW_SECURITY_HMAC                        0x6917
+    #define SW_SECURITY_RANGE_VALUE                 0x6918
+    #define SW_SECURITY_INTERNAL                    0x6919
+    #define SW_SECURITY_MAX_SIGNATURE_REACHED       0x691A
+    #define SW_SECURITY_PREFIX_HASH                 0x691B
+    #define SW_SECURITY_LOCKED                      0x69EE
+    #define SW_COMMAND_NOT_ALLOWED                  0x6980
+    #define SW_SUBCOMMAND_NOT_ALLOWED               0x6981
+    #define SW_DENY                                 0x6982
+    #define SW_KEY_NOT_SET                          0x6983
+    #define SW_WRONG_DATA                           0x6984
+    #define SW_WRONG_DATA_RANGE                     0x6985
+    #define SW_IO_FULL                              0x6986
+    #define SW_CLIENT_NOT_SUPPORTED                 0x6A30
+    #define SW_WRONG_P1P2                           0x6b00
+    #define SW_INS_NOT_SUPPORTED                    0x6d00
+    #define SW_PROTOCOL_NOT_SUPPORTED               0x6e00
+    #define SW_UNKNOWN                              0x6f00
 
     namespace {
         bool apdu_verbose =true;