Fix parse devices to not include invalid device

Signed-off-by: Kathryn Baldauf <kabaldau@microsoft.com>
microsoft · Sep 27, 2024 · ffe8282 · ffe8282
1 parent e1b4506
commit ffe8282
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/internal/oci/uvm.go b/internal/oci/uvm.go
@@ -231,12 +231,12 @@ func parseDevices(ctx context.Context, specWindows *specs.Windows) []uvm.VPCIDev
 	if specWindows == nil || specWindows.Devices == nil {
 		return nil
 	}
-	extraDevices := make([]uvm.VPCIDeviceID, len(specWindows.Devices))
-	for i, d := range specWindows.Devices {
+	extraDevices := []uvm.VPCIDeviceID{}
+	for _, d := range specWindows.Devices {
 		pciID, index := devices.GetDeviceInfoFromPath(d.ID)
 		if uvm.IsValidDeviceType(d.IDType) {
 			key := uvm.NewVPCIDeviceID(pciID, index)
-			extraDevices[i] = key
+			extraDevices = append(extraDevices, key)
 		} else {
 			log.G(ctx).WithFields(logrus.Fields{
 				"device": d,