diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2015-01-28 16:17:36 +0300 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2015-01-28 13:42:12 +0000 |
| commit | 8bc30e07338d8a8d4a972261e4ff25fa85cec921 (patch) | |
| tree | f9cfa04d311ffc6e9f0a8b0b2818995e0f8fd910 | |
| parent | 67f8a8131648346f6bf9b525cd989bd2f7293b3f (diff) | |
| download | go-8bc30e07338d8a8d4a972261e4ff25fa85cec921.tar.xz | |
net/http: fix goroutine leak in benchmark
Race builders report goroutine leaks after addition of this benchmark:
http://build.golang.org/log/18e47f4cbc18ee8db125e1f1157573dd1e333c41
Close idle connection in default transport.
Change-Id: I86ff7b2e0972ed47c5ebcb9fce19e7f39d3ff530
Reviewed-on: https://go-review.googlesource.com/3412
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
| -rw-r--r-- | src/net/http/serve_test.go | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/net/http/serve_test.go b/src/net/http/serve_test.go index 8086ef0800..6839865658 100644 --- a/src/net/http/serve_test.go +++ b/src/net/http/serve_test.go @@ -2977,6 +2977,7 @@ func BenchmarkClient(b *testing.B) { case <-time.After(5 * time.Second): b.Fatalf("subprocess did not stop") } + DefaultTransport.(*Transport).CloseIdleConnections() } func BenchmarkServerFakeConnNoKeepAlive(b *testing.B) { |