Resolve Windows agent crash issue caused by egressSNATRandomFully

Signed-off-by: Shuyang Xin <[email protected]>
antrea-io · Sep 12, 2024 · e219dc6 · e219dc6
1 parent afdd261
commit e219dc6
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/cmd/antrea-agent/options.go b/cmd/antrea-agent/options.go
@@ -493,9 +493,11 @@ func (o *Options) setK8sNodeDefaultOptions() {
 		if o.config.Egress.MaxEgressIPsPerNode == 0 {
 			o.config.Egress.MaxEgressIPsPerNode = defaultMaxEgressIPsPerNode
 		}
-		if o.config.Egress.SNATFullyRandomPorts == nil {
-			o.config.Egress.SNATFullyRandomPorts = ptr.To(o.config.SNATFullyRandomPorts)
-		}
+	}
+
+	// Regardless of whether the egress feature is enabled, o.config.Egress.SNATFullyRandomPorts should not be nil to prevent a crash in NewClient().
+	if o.config.Egress.SNATFullyRandomPorts == nil {
+		o.config.Egress.SNATFullyRandomPorts = ptr.To(o.config.SNATFullyRandomPorts)
 	}
 }