utils: simplify newProcess

This code is not in libcontainer, IOW it is only used by a short lived binary (runc start/run/exec). Unlike libcontainer code, here we don't have to care about copying the structures supplied as input, meaning we can just reuse the pointers directly. Signed-off-by: Kir Kolyshkin <[email protected]>
opencontainers · Jun 26, 2024 · 075da0a · 075da0a
1 parent 593fcf9
commit 075da0a
Showing 1 changed file with 2 additions and 10 deletions.
diff --git a/utils_linux.go b/utils_linux.go
@@ -55,23 +55,15 @@ func newProcess(p specs.Process) (*libcontainer.Process, error) {
 		Label:           p.SelinuxLabel,
 		NoNewPrivileges: &p.NoNewPrivileges,
 		AppArmorProfile: p.ApparmorProfile,
+		Scheduler:       p.Scheduler,
+		IOPriority:      p.IOPriority,
 	}
 
 	if p.ConsoleSize != nil {
 		lp.ConsoleWidth = uint16(p.ConsoleSize.Width)
 		lp.ConsoleHeight = uint16(p.ConsoleSize.Height)
 	}
 
-	if p.Scheduler != nil {
-		s := *p.Scheduler
-		lp.Scheduler = &s
-	}
-
-	if p.IOPriority != nil {
-		ioPriority := *p.IOPriority
-		lp.IOPriority = &ioPriority
-	}
-
 	if p.Capabilities != nil {
 		lp.Capabilities = &configs.Capabilities{}
 		lp.Capabilities.Bounding = p.Capabilities.Bounding