Add comment

NihaNallappagari · NihaNallappagari · commit ecf933e4df1b · 2025-09-19T15:06:22.000-05:00
diff --git a/azure-ipam/ipam.go b/azure-ipam/ipam.go
@@ -137,7 +137,8 @@ func (p *IPAMPlugin) CmdAdd(args *cniSkel.CmdArgs) error {
 			}
 		}
 
-		if gatewayIP != nil && len(*gatewayIP) > i {
+		// Check gateway index before assigning to avoid index out of bounds exception and log warning for nil gateway
+		if len(*gatewayIP) > i && (*gatewayIP)[i] != nil {
 			ipConfig.Gateway = (*gatewayIP)[i]
 		} else {
 			p.logger.Warn("No gateway IP returned from CNS, setting to nil", zap.Any("gatewayIP", gatewayIP), zap.Int("index", i))

Original file line number	Diff line number	Diff line change
`@@ -137,7 +137,8 @@ func (p IPAMPlugin) CmdAdd(args cniSkel.CmdArgs) error {`
`137`	`137`	`}`
`138`	`138`	`}`
`139`	`139`
`140`		`- if gatewayIP != nil && len(*gatewayIP) > i {`
	`140`	`+ // Check gateway index before assigning to avoid index out of bounds exception and log warning for nil gateway`
	`141`	`+ if len(gatewayIP) > i && (gatewayIP)[i] != nil {`
`141`	`142`	`ipConfig.Gateway = (*gatewayIP)[i]`
`142`	`143`	`} else {`
`143`	`144`	`p.logger.Warn("No gateway IP returned from CNS, setting to nil", zap.Any("gatewayIP", gatewayIP), zap.Int("index", i))`