From cd16be9308d0886a1fe2b5e1bb5e0bc935cd9217 Mon Sep 17 00:00:00 2001
From: Jack Christensen <jack@jackchristensen.com>
Date: Thu, 8 Jun 2017 19:14:03 -0500
Subject: [PATCH] Close database/sql connections created through ConnPool

database/sql driver created through stdlib.OpenFromConnPool closes connections
when requested by database/sql rather than release to underlying connection
pool.
---
 CHANGELOG.md  | 6 ++++++
 stdlib/sql.go | 4 ++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 2fa56bbc..88c663b0 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,9 @@
+# Unreleased
+
+## Fixes
+
+* database/sql driver created through stdlib.OpenFromConnPool closes connections when requested by database/sql rather than release to underlying connection pool.
+
 # 2.11.0 (June 5, 2017)
 
 ## Fixes
diff --git a/stdlib/sql.go b/stdlib/sql.go
index 1c13e70c..8c78cd39 100644
--- a/stdlib/sql.go
+++ b/stdlib/sql.go
@@ -180,12 +180,12 @@ func (c *Conn) Prepare(query string) (driver.Stmt, error) {
 }
 
 func (c *Conn) Close() error {
+	err := c.conn.Close()
 	if c.pool != nil {
 		c.pool.Release(c.conn)
-		return nil
 	}
 
-	return c.conn.Close()
+	return err
 }
 
 func (c *Conn) Begin() (driver.Tx, error) {