diff options
author | Daniel Theophanes <kardianos@gmail.com> | 2017-02-08 21:38:51 -0800 |
---|---|---|
committer | Russ Cox <rsc@golang.org> | 2017-02-10 17:53:36 +0000 |
commit | d84dee069a534bd7a35ec4062a92a04ce9c2806e (patch) | |
tree | cbae73613378b5f57516891330ab17eed3e8fc8a | |
parent | f1e44a4b7426e03bff7ed4f75a91e57c498d3463 (diff) | |
download | go-d84dee069a534bd7a35ec4062a92a04ce9c2806e.tar.gz go-d84dee069a534bd7a35ec4062a92a04ce9c2806e.zip |
[release-branch.go1.8] database/sql: ensure driverConns are closed if not returned to pool
Previously if a connection was requested but timed out during the
request and when acquiring the db.Lock the connection request
is fulfilled and the request is unable to be returned to the
connection pool, then then driver connection would not be closed.
No tests were added or modified because I was unable to determine
how to trigger this situation without something invasive.
Change-Id: I9d4dc680e3fdcf63d79d212174a5b8b313f363f1
Reviewed-on: https://go-review.googlesource.com/36641
Reviewed-by: Russ Cox <rsc@golang.org>
Reviewed-on: https://go-review.googlesource.com/36714
Run-TryBot: Russ Cox <rsc@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-rw-r--r-- | src/database/sql/sql.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/database/sql/sql.go b/src/database/sql/sql.go index 1e27b17cf9..c016681fca 100644 --- a/src/database/sql/sql.go +++ b/src/database/sql/sql.go @@ -939,14 +939,14 @@ func (db *DB) conn(ctx context.Context, strategy connReuseStrategy) (*driverConn // on it after removing. db.mu.Lock() delete(db.connRequests, reqKey) + db.mu.Unlock() select { default: case ret, ok := <-req: if ok { - db.putConnDBLocked(ret.conn, ret.err) + db.putConn(ret.conn, ret.err) } } - db.mu.Unlock() return nil, ctx.Err() case ret, ok := <-req: if !ok { |