net/http: fix minor leak in Header.WriteSubset

Header.WriteSubset uses a sync.Pool but wouldn't Put the sorter back in
the pool if there was an error writing to the io.Writer

I'm not really sure why the sorter is returned to begin with. The
comment says "for possible return to headerSorterCache".

This also doesn't address potential panics that might occur, but the
overhead of doing the Put in a defer would likely be too great.

Change-Id: If3c45a4c3e11f6ec65d187e25b63455b0142d4e3
Reviewed-on: https://go-review.googlesource.com/73910
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Tom Bergan <tombergan@google.com>

diff --git a/src/net/http/header.go b/src/net/http/header.go
index 832169247fed7ced1de930938cfa620265925abb..622ad289636ed12796dd9ace05032584660c7ea8 100644 (file)
--- a/src/net/http/header.go
+++ b/src/net/http/header.go
@@ -156,6 +156,7 @@ func (h Header) WriteSubset(w io.Writer, exclude map[string]bool) error {
                        v = textproto.TrimString(v)
                        for _, s := range []string{kv.key, ": ", v, "\r\n"} {
                                if _, err := ws.WriteString(s); err != nil {
+                                       headerSorterPool.Put(sorter)
                                        return err
                                }
                        }