From 87a379f6631e3986efc0513d38cfdd3bb8a9c0d0 Mon Sep 17 00:00:00 2001 From: Oleksii Kliukin Date: Thu, 10 Aug 2017 18:19:35 +0200 Subject: [PATCH] Avoid reusing closed DB connection. (#79) Set DB connection to nil upon closing it. --- pkg/cluster/pg.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/pkg/cluster/pg.go b/pkg/cluster/pg.go index e5712952e..0fd94f7d1 100644 --- a/pkg/cluster/pg.go +++ b/pkg/cluster/pg.go @@ -64,7 +64,10 @@ func (c *Cluster) initDbConn() (err error) { func (c *Cluster) closeDbConn() (err error) { if c.pgDb != nil { c.logger.Debug("closing database connection") - return c.pgDb.Close() + if err = c.pgDb.Close(); err != nil { + c.logger.Errorf("could not close database connection: %v", err) + } + c.pgDb = nil } c.logger.Warning("attempted to close an empty db connection object") return nil