From aeec20018cd06ea7fb1a8b04699f3251aa6d29c6 Mon Sep 17 00:00:00 2001 From: Denis Arh Date: Wed, 25 May 2022 15:48:49 +0200 Subject: [PATCH] Prevent nil pointer dereference inside corredor healthcheck --- pkg/corredor/healthcheck.go | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/pkg/corredor/healthcheck.go b/pkg/corredor/healthcheck.go index bd10edd00..5b0dc5fbf 100644 --- a/pkg/corredor/healthcheck.go +++ b/pkg/corredor/healthcheck.go @@ -19,6 +19,13 @@ func Healthcheck(_ context.Context) error { return nil } + if svc.conn == nil { + // working around edge-case where health-check is called + // but conn variable is not set; + // prevents nil pointer dereference error + return nil + } + if state := svc.conn.GetState(); state != connectivity.Ready { return fmt.Errorf("connection is %s", state) }