BUG/MEDIUM: configuration: Fix nil pointer dereference

mahnkong · mjuraga · commit 83b7655422e1 · 2023-03-14T10:57:34.000+01:00
diff --git a/dataplaneapi_configuration.go b/dataplaneapi_configuration.go
@@ -35,7 +35,7 @@ func SyncWithFileSettings(server *Server, cfg *configuration.Configuration) { //
 			configStorage.Dataplaneapi.EnabledListeners = &[]string{"http"}
 		}
 	}
-	if configStorage.Dataplaneapi.EnabledListeners != nil && !misc.HasOSArg("", "scheme", "") {
+	if configStorage.Dataplaneapi != nil && configStorage.Dataplaneapi.EnabledListeners != nil && !misc.HasOSArg("", "scheme", "") {
 		server.EnabledListeners = *configStorage.Dataplaneapi.EnabledListeners
 	}
 	if configStorage.Dataplaneapi != nil && configStorage.Dataplaneapi.CleanupTimeout != nil && !misc.HasOSArg("", "cleanup-timeout", "") {

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ func SyncWithFileSettings(server Server, cfg configuration.Configuration) { //`
`35`	`35`	`configStorage.Dataplaneapi.EnabledListeners = &[]string{"http"}`
`36`	`36`	`}`
`37`	`37`	`}`
`38`		`- if configStorage.Dataplaneapi.EnabledListeners != nil && !misc.HasOSArg("", "scheme", "") {`
	`38`	`+ if configStorage.Dataplaneapi != nil && configStorage.Dataplaneapi.EnabledListeners != nil && !misc.HasOSArg("", "scheme", "") {`
`39`	`39`	`server.EnabledListeners = *configStorage.Dataplaneapi.EnabledListeners`
`40`	`40`	`}`
`41`	`41`	`if configStorage.Dataplaneapi != nil && configStorage.Dataplaneapi.CleanupTimeout != nil && !misc.HasOSArg("", "cleanup-timeout", "") {`