code review 5477092: net/http: fix bug in error checking

net/http: fix bug in error checking

Thanks to josef86@gmail.com for pointing this out.

[gri, 2011-12-14 16:37:48]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://go.googlecode.com/hg/

[iant2, 2011-12-14 16:42:11]

LGTM

gri@golang.org writes:

> Reviewers: golang-dev_googlegroups.com,
>
> Message:
> Hello golang-dev@googlegroups.com,
>
> I'd like you to review this change to
> https://go.googlecode.com/hg/
>
>
> Description:
> net/http: fix bug in error checking
>
> Thanks to josef86@gmail.com for pointing this out.
>
> Please review this at http://codereview.appspot.com/5477092/
>
> Affected files:
>   M src/pkg/net/http/transport.go
>
>
> Index: src/pkg/net/http/transport.go
> ===================================================================
> --- a/src/pkg/net/http/transport.go
> +++ b/src/pkg/net/http/transport.go
> @@ -544,7 +544,7 @@
>  				resp.Header.Del("Content-Length")
>  				resp.ContentLength = -1
>  				gzReader, zerr := gzip.NewReader(resp.Body)
> -				if err != nil {
> +				if zerr != nil {
>  					pc.close()
>  					err = zerr
>  				} else {

[gri, 2011-12-14 16:43:46]

*** Submitted as http://code.google.com/p/go/source/detail?r=c3c6e72d7cc5 ***

net/http: fix bug in error checking

Thanks to josef86@gmail.com for pointing this out.

R=golang-dev, iant
CC=golang-dev
http://codereview.appspot.com/5477092