From 785cc739ee009b40fa9da86a5864a922ff9b94d9 Mon Sep 17 00:00:00 2001 From: mayor Date: Wed, 14 Jan 2026 16:22:42 +0100 Subject: [PATCH] Fix nil pointer in NewSearchServer - use serverOptions.Backend The backend may be created inside the function and assigned to serverOptions.Backend, not opts.Backend. Using opts.Backend caused a nil pointer dereference in searchSupport.buildIndexes. Co-Authored-By: Claude Opus 4.5 --- pkg/storage/unified/sql/server.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/storage/unified/sql/server.go b/pkg/storage/unified/sql/server.go index 046a6e0848a..a8baaca6fe2 100644 --- a/pkg/storage/unified/sql/server.go +++ b/pkg/storage/unified/sql/server.go @@ -159,7 +159,7 @@ func NewResourceServer(opts ServerOptions) (resource.ResourceServer, resource.Se } } - search, err := resource.NewSearchServer(opts.SearchOptions, opts.Backend, opts.AccessClient, nil, opts.IndexMetrics, opts.OwnsIndexFn) + search, err := resource.NewSearchServer(opts.SearchOptions, serverOptions.Backend, opts.AccessClient, nil, opts.IndexMetrics, opts.OwnsIndexFn) if err != nil { return nil, nil, fmt.Errorf("failed to initialize search: %w", err) }