Merge pull request #152 from LedgerHQ/151-bolos-specific-apdus-are-no…

…t-managed Manage BOLOS default APDUs
LedgerHQ · Apr 10, 2024 · dfa0725 · dfa0725
2 parents a882f1d + 79a718d
commit dfa0725
Show file tree

Hide file tree

Showing 3 changed files with 48 additions and 2 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/ledger_device_sdk/Cargo.toml b/ledger_device_sdk/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "ledger_device_sdk"
-version = "1.8.0"
+version = "1.8.1"
 authors = ["yhql", "yogh333"]
 edition = "2021"
 license.workspace = true

diff --git a/ledger_device_sdk/src/io.rs b/ledger_device_sdk/src/io.rs
@@ -331,6 +331,15 @@ impl Comm {
                 return None;
             }
 
+            // Manage BOLOS specific APDUs B0xx0000
+            if self.apdu_buffer[0] == 0xB0
+                && self.apdu_buffer[2] == 0x00
+                && self.apdu_buffer[3] == 0x00
+            {
+                handle_bolos_apdu(self, self.apdu_buffer[1]);
+                return None;
+            }
+
             // If CLA filtering is enabled, automatically reject APDUs with wrong CLA
             if let Some(cla) = self.expected_cla {
                 if self.apdu_buffer[0] != cla {
@@ -467,6 +476,43 @@ impl Comm {
     }
 }
 
+// BOLOS APDU Handling (see https://developers.ledger.com/docs/connectivity/ledgerJS/open-close-info-on-apps)
+fn handle_bolos_apdu(com: &mut Comm, ins: u8) {
+    match ins {
+        // Get Information INS: retrieve App name and version
+        0x01 => {
+            unsafe {
+                com.apdu_buffer[0] = 0x01;
+                com.tx += 1;
+                let len = os_registry_get_current_app_tag(
+                    BOLOS_TAG_APPNAME,
+                    &mut com.apdu_buffer[com.tx + 1] as *mut u8,
+                    (260 - com.tx - 1) as u32,
+                );
+                com.apdu_buffer[com.tx] = len as u8;
+                com.tx += (1 + len) as usize;
+
+                let len = os_registry_get_current_app_tag(
+                    BOLOS_TAG_APPVERSION,
+                    &mut com.apdu_buffer[com.tx + 1] as *mut u8,
+                    (260 - com.tx - 1) as u32,
+                );
+                com.apdu_buffer[com.tx] = len as u8;
+                com.tx += (1 + len) as usize;
+            }
+            com.reply_ok();
+        }
+        // Quit Application INS
+        0xa7 => {
+            com.reply_ok();
+            crate::exit_app(0);
+        }
+        _ => {
+            com.reply(StatusWords::BadIns);
+        }
+    }
+}
+
 impl Index<usize> for Comm {
     type Output = u8;
     fn index(&self, idx: usize) -> &Self::Output {