From bd8a04a6a573e450d1feb57dff76a9f0f58d2f1f Mon Sep 17 00:00:00 2001 From: Marcus Efraimsson Date: Fri, 12 Jun 2020 16:04:09 +0200 Subject: [PATCH] InfluxDB: Fix invalid memory address or nil pointer dereference when schema is missing in URL (#25565) Add guard if parsing url fails in influxdb backend data source. Fixes #25242 --- pkg/tsdb/influxdb/influxdb.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/pkg/tsdb/influxdb/influxdb.go b/pkg/tsdb/influxdb/influxdb.go index 9d8e98bb659..2b76c15f59b 100644 --- a/pkg/tsdb/influxdb/influxdb.go +++ b/pkg/tsdb/influxdb/influxdb.go @@ -141,7 +141,10 @@ func (e *InfluxDBExecutor) getQuery(dsInfo *models.DataSource, queries []*tsdb.Q func (e *InfluxDBExecutor) createRequest(dsInfo *models.DataSource, query string) (*http.Request, error) { - u, _ := url.Parse(dsInfo.Url) + u, err := url.Parse(dsInfo.Url) + if err != nil { + return nil, err + } u.Path = path.Join(u.Path, "query") httpMode := dsInfo.JsonData.Get("httpMode").MustString("GET")