Avoid reusing closed DB connection.

Set DB connection to nil upon closing it.
zalando · Aug 10, 2017 · 7871158 · 7871158
1 parent c314b9f
commit 7871158
Showing 1 changed file with 4 additions and 1 deletion.
diff --git 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