From 922aef3e5a642a4d3ee7323e49e1bac066902c9a Mon Sep 17 00:00:00 2001 From: Maista <131594657+Maista6969@users.noreply.github.com> Date: Thu, 24 Aug 2023 04:32:12 +0200 Subject: [PATCH] Fix nil pointer dereference (#4051) This could happen when a user had configured tag exclusions and a scraper returned an empty result --- internal/api/resolver_query_scraper.go | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/internal/api/resolver_query_scraper.go b/internal/api/resolver_query_scraper.go index 7b7694341..0220316b2 100644 --- a/internal/api/resolver_query_scraper.go +++ b/internal/api/resolver_query_scraper.go @@ -129,7 +129,9 @@ func (r *queryResolver) ScrapeScene(ctx context.Context, scraperID string, scene return nil, err } - filterSceneTags([]*scraper.ScrapedScene{ret}) + if ret != nil { + filterSceneTags([]*scraper.ScrapedScene{ret}) + } return ret, nil } @@ -190,7 +192,9 @@ func (r *queryResolver) ScrapeSceneURL(ctx context.Context, url string) (*scrape return nil, err } - filterSceneTags([]*scraper.ScrapedScene{ret}) + if ret != nil { + filterSceneTags([]*scraper.ScrapedScene{ret}) + } return ret, nil }