group1: cpusetController.getValues: don't use naked returns

Explicitly return nil instead of potentially returning a partial result (only cpus or mems). Signed-off-by: Sebastiaan van Stijn <[email protected]>
containerd · Dec 10, 2024 · 4fc13cd · 4fc13cd
1 parent d4e976d
commit 4fc13cd
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/cgroup1/cpuset.go b/cgroup1/cpuset.go
@@ -87,10 +87,10 @@ func (c *cpusetController) Update(path string, resources *specs.LinuxResources)
 
 func (c *cpusetController) getValues(path string) (cpus []byte, mems []byte, err error) {
 	if cpus, err = os.ReadFile(filepath.Join(path, "cpuset.cpus")); err != nil && !os.IsNotExist(err) {
-		return
+		return nil, nil, err
 	}
 	if mems, err = os.ReadFile(filepath.Join(path, "cpuset.mems")); err != nil && !os.IsNotExist(err) {
-		return
+		return nil, nil, err
 	}
 	return cpus, mems, nil
 }