commit 05d3205156cb1bdc096578d6457fa161cd950aa2
author Chris Roche <rodaine@gmail.com> Tue Mar 28 17:59:59 2017 -0700
committer Tom Bergan <tombergan@google.com> Wed Mar 29 04:46:32 2017 +0000
tree f4b9c98e085a85ab5b0306ec8c03fc4228adce50
parent c4f37062e123971566edb19fb0554351464f8a98

http2/hpack: fix memory leak in headerFieldTable lookup maps

An earlier performance change, https://golang.org/cl/37406,
made headerFieldTable lookups O(1). The entries in the maps used to perform
these lookups were not evicted along with the original field elements, however
causing a gradual memory leak. This is most apparent when the headers have
highly variable content such as request IDs or timings.

Fixes golang/go#19756

Change-Id: Icdb51527eb671925216350ada15f2a1336ea3158
Reviewed-on: https://go-review.googlesource.com/38781
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Tom Bergan <tombergan@google.com>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>