Fix implicit memory aliasing in loop

Signed-off-by: Evan Lezar <[email protected]>
NVIDIA · Jan 28, 2024 · de81383 · de81383
1 parent b25e906
commit de81383
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/cmd/nvidia-mig-parted/assert/assert.go b/cmd/nvidia-mig-parted/assert/assert.go
@@ -234,7 +234,8 @@ func WalkSelectedMigConfigForEachGPU(migConfig v1.MigConfigSpecSlice, f func(*v1
 
 			log.Debugf("  GPU %v: %v", i, deviceID)
 
-			err = f(&mc, i, deviceID)
+			migConfigSpec := mc
+			err = f(&migConfigSpec, i, deviceID)
 			if err != nil {
 				return err
 			}

diff --git a/pkg/mig/state/state.go b/pkg/mig/state/state.go
@@ -216,7 +216,8 @@ func (m *migStateManager) RestoreConfig(state *types.MigState) error {
 				return fmt.Errorf("error getting GPU instance profile info for '%v': %v", giState.ProfileID, ret)
 			}
 
-			gi, ret := device.CreateGpuInstanceWithPlacement(&giProfileInfo, &giState.Placement)
+			placement := giState.Placement
+			gi, ret := device.CreateGpuInstanceWithPlacement(&giProfileInfo, &placement)
 			if ret.Value() != nvml.SUCCESS {
 				return fmt.Errorf("error creating GPU instance for '%v': %v", giState.ProfileID, ret)
 			}