[VM Agent] Fix agent crash issue with nil pointer exception (#6748)

Set default value on config.Egress.SNATFullyRandomPorts with VM Agent scenario to avoid agent crash, although routeClient is not really consumed. Fixes #6747 Signed-off-by: Wenying Dong <[email protected]>
antrea-io · Oct 16, 2024 · d6f809f · d6f809f
1 parent 9fd5d83
commit d6f809f
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/cmd/antrea-agent/options.go b/cmd/antrea-agent/options.go
@@ -690,6 +690,10 @@ func (o *Options) setExternalNodeDefaultOptions() {
 	if o.config.ExternalNode.ExternalNodeNamespace == "" {
 		o.config.ExternalNode.ExternalNodeNamespace = "default"
 	}
+	// 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)
+	}
 }
 
 func (o *Options) setMulticlusterDefaultOptions() {