Update to heapless 0.9.1 and iso7816 0.2.0

Author: sosthene-nitrokey

Cargo.toml

@@ -9,7 +9,9 @@ license = "Apache-2.0 OR MIT"
 repository = "https://github.com/trussed-dev/apdu-dispatch"
 
 [workspace.dependencies]
-iso7816 = "0.1.2"
+iso7816 = "0.2.0"
+heapless = "0.9"
 
 [patch.crates-io]
 apdu-app.path = "app"
+iso7816 = { git = "https://github.com/trussed-dev/iso7816.git", branch = "heapless-09"}

app/Cargo.toml

@@ -10,3 +10,4 @@ repository.workspace = true
 
 [dependencies]
 iso7816.workspace = true
+heapless.workspace = true

app/src/lib.rs

@@ -1,19 +1,20 @@
 #![no_std]
 
-pub use iso7816::{command::CommandView, Data, Interface, Status};
+use heapless::VecView;
+pub use iso7816::{command::CommandView, Interface, Status};
 
 pub type Result = iso7816::Result<()>;
 
 /// An App can receive and respond APDUs at behest of the ApduDispatch.
-pub trait App<const R: usize>: iso7816::App {
+pub trait App: iso7816::App {
     /// Given parsed APDU for select command.
     /// Write response data back to buf, and return length of payload.  Return APDU Error code on error.
     /// Alternatively, the app can defer the response until later by returning it in `poll()`.
     fn select(
         &mut self,
         interface: Interface,
         apdu: CommandView<'_>,
-        reply: &mut Data<R>,
+        reply: &mut VecView<u8>,
     ) -> Result;
 
     /// Deselects the app. This is the result of another app getting selected.
@@ -22,5 +23,10 @@ pub trait App<const R: usize>: iso7816::App {
 
     /// Given parsed APDU for app when selected.
     /// Write response data back to buf, and return length of payload.  Return APDU Error code on error.
-    fn call(&mut self, interface: Interface, apdu: CommandView<'_>, reply: &mut Data<R>) -> Result;
+    fn call(
+        &mut self,
+        interface: Interface,
+        apdu: CommandView<'_>,
+        reply: &mut VecView<u8>,
+    ) -> Result;
 }

dispatch/Cargo.toml

@@ -11,9 +11,9 @@ repository.workspace = true
 [dependencies]
 apdu-app = "0.1.0"
 delog = "0.1.4"
-heapless = "0.7"
 interchange = "0.3.0"
 iso7816.workspace = true
+heapless.workspace = true
 
 [dev-dependencies]
 # Testing

dispatch/src/dispatch.rs

@@ -123,8 +123,8 @@ impl<'pipe> ApduDispatch<'pipe> {
     // but that won't work due to ownership rules
     fn find_app<'a, 'b>(
         aid: Option<&Aid>,
-        apps: &'a mut [&'b mut dyn App<ResponseSize>],
-    ) -> Option<&'a mut &'b mut dyn App<ResponseSize>> {
+        apps: &'a mut [&'b mut dyn App],
+    ) -> Option<&'a mut &'b mut dyn App> {
         // match aid {
         //     Some(aid) => apps.iter_mut().find(|app| aid.starts_with(app.rid())),
         //     None => None,
@@ -366,12 +366,7 @@ impl<'pipe> ApduDispatch<'pipe> {
     }
 
     #[inline(never)]
-    fn handle_app_select(
-        &mut self,
-        apps: &mut [&mut dyn App<ResponseSize>],
-        aid: Aid,
-        interface: Interface,
-    ) {
+    fn handle_app_select(&mut self, apps: &mut [&mut dyn App], aid: Aid, interface: Interface) {
         // three cases:
         // - currently selected app has different AID -> deselect it, to give it
         //   the chance to clear sensitive state
@@ -412,11 +407,7 @@ impl<'pipe> ApduDispatch<'pipe> {
     }
 
     #[inline(never)]
-    fn handle_app_command(
-        &mut self,
-        apps: &mut [&mut dyn App<ResponseSize>],
-        interface: Interface,
-    ) {
+    fn handle_app_command(&mut self, apps: &mut [&mut dyn App], interface: Interface) {
         // if there is a selected app, send it the command
         let mut response = response::Data::new();
         if let Some(app) = Self::find_app(self.current_aid.as_ref(), apps) {
@@ -431,7 +422,7 @@ impl<'pipe> ApduDispatch<'pipe> {
         };
     }
 
-    pub fn poll(&mut self, apps: &mut [&mut dyn App<ResponseSize>]) -> Option<Interface> {
+    pub fn poll(&mut self, apps: &mut [&mut dyn App]) -> Option<Interface> {
         // Only take on one transaction at a time.
         let request_type = self.check_for_request();
 

dispatch/tests/dispatch.rs

@@ -1,6 +1,7 @@
 use apdu_dispatch::app::{App, CommandView, Result as AppResult};
 use apdu_dispatch::dispatch;
 use apdu_dispatch::{interchanges, response};
+use heapless::VecView;
 use hex_literal::hex;
 use interchange::Channel;
 use iso7816::Status;
@@ -47,12 +48,12 @@ impl iso7816::App for TestApp1 {
 }
 
 // This app echos to Ins code 0x10
-impl App<{ apdu_dispatch::response::SIZE }> for TestApp1 {
+impl App for TestApp1 {
     fn select(
         &mut self,
         _interface: dispatch::Interface,
         _apdu: CommandView<'_>,
-        _reply: &mut response::Data,
+        _reply: &mut VecView<u8>,
     ) -> AppResult {
         Ok(())
     }
@@ -63,7 +64,7 @@ impl App<{ apdu_dispatch::response::SIZE }> for TestApp1 {
         &mut self,
         _: dispatch::Interface,
         apdu: CommandView<'_>,
-        reply: &mut response::Data,
+        reply: &mut VecView<u8>,
     ) -> AppResult {
         println!("TestApp1::call");
         match apdu.instruction().into() {
@@ -79,8 +80,8 @@ impl App<{ apdu_dispatch::response::SIZE }> for TestApp1 {
             }
             // For measuring the stack burden of dispatch
             0x15 => {
-                let buf = heapless::Vec::new();
-                let addr = (&buf as *const response::Data) as u32;
+                let buf = heapless::Vec::<u8, { response::SIZE }>::new();
+                let addr = (&buf as *const VecView<u8>).addr() as u32;
                 reply.extend_from_slice(&addr.to_be_bytes()).unwrap();
                 Ok(())
             }
@@ -122,12 +123,12 @@ impl iso7816::App for TestApp2 {
 }
 
 // This app echos to Ins code 0x20
-impl App<{ apdu_dispatch::response::SIZE }> for TestApp2 {
+impl App for TestApp2 {
     fn select(
         &mut self,
         _interface: dispatch::Interface,
         _apdu: CommandView<'_>,
-        _reply: &mut response::Data,
+        _reply: &mut VecView<u8>,
     ) -> AppResult {
         Ok(())
     }
@@ -138,7 +139,7 @@ impl App<{ apdu_dispatch::response::SIZE }> for TestApp2 {
         &mut self,
         _: dispatch::Interface,
         apdu: CommandView<'_>,
-        reply: &mut response::Data,
+        reply: &mut VecView<u8>,
     ) -> AppResult {
         println!("TestApp2::call");
         match apdu.instruction().into() {
@@ -174,12 +175,12 @@ impl iso7816::App for PanicApp {
 }
 
 // This app echos to Ins code 0x20
-impl App<{ apdu_dispatch::response::SIZE }> for PanicApp {
+impl App for PanicApp {
     fn select(
         &mut self,
         _interface: dispatch::Interface,
         _apdu: CommandView<'_>,
-        _reply: &mut response::Data,
+        _reply: &mut VecView<u8>,
     ) -> AppResult {
         panic!("Dont call the panic app");
     }
@@ -192,7 +193,7 @@ impl App<{ apdu_dispatch::response::SIZE }> for PanicApp {
         &mut self,
         _: dispatch::Interface,
         _apdu: CommandView<'_>,
-        _reply: &mut response::Data,
+        _reply: &mut VecView<u8>,
     ) -> AppResult {
         panic!("Dont call the panic app");
     }