github/sparrow-kata-full
1
0
Fork 0
mirror of https://github.com/AmbiML/sparrow-kata-full.git synced 2025-08-13 10:56:16 +00:00
Code Issues Projects Releases Wiki Activity

kata-proc-manager: small code shuffle

Browse Source 
Move the container slot release up as it logically belongs with the
seL4BundleImpl construction. This makes clear it's unrelated to the
actual start operation.

Change-Id: Idda2a4a829fe6fae2a4d1dbe99aff495ef10b3b8
GitOrigin-RevId: ba2cdc9f4bfb54cfbeca11888bd31cd5657d0182
This commit is contained in:
Sam Leffler 2022-06-23 03:15:49 +00:00
parent b0d1b6efac
commit 272c18cf9b
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
apps/system/components/ProcessManager/kata-proc-manager/src/lib.rs
View File

@ -141,12 +141,12 @@ impl ProcessManagerInterface for KataManagerInterface {
let bundle_frames = let bundle_frames =
kata_security_load_application(&bundle.app_id, &container_slot)?; kata_security_load_application(&bundle.app_id, &container_slot)?;
let mut sel4_bundle = seL4BundleImpl::new(bundle, &bundle_frames)?; let mut sel4_bundle = seL4BundleImpl::new(bundle, &bundle_frames)?;
sel4_bundle.start()?;
// sel4_bundle owns container_slot now; release our ref so it's not // sel4_bundle owns container_slot now; release our ref so it's not
// reclaimed when container_slot goes out of scope. // reclaimed when container_slot goes out of scope.
container_slot.release(); container_slot.release();
sel4_bundle.start()?;
Ok(Box::new(sel4_bundle) as _) Ok(Box::new(sel4_bundle) as _)
} }
fn stop(&mut self, bundle_impl: &mut dyn BundleImplInterface) -> Result<(), ProcessManagerError> { fn stop(&mut self, bundle_impl: &mut dyn BundleImplInterface) -> Result<(), ProcessManagerError> {