[release-branch.go1] net: fix race between Close and Read

««« backport 5f24ff99b5f1
net: fix race between Close and Read

Fixes #3507.

Applied the suggested fix from rsc. If the connection
is in closing state then errClosing will bubble up to
the caller.

The fix has been applied to udp, ip and unix as well as
their code path include nil'ing c.fd on close. Func
tests are available in the linked issue that verified
the bug existed there as well.

R=rsc, fullung, alex.brainman, mikioh.mikioh
CC=golang-dev
https://golang.org/cl/6002053
»»»

src/pkg/net/iprawsock_posix.go

@@ -83,9 +83,7 @@ func (c *IPConn) Close() error {
 	if !c.ok() {
 		return syscall.EINVAL
 	}
-	err := c.fd.Close()
+	return c.fd.Close()
-	c.fd = nil
-	return err
 }
 
 // LocalAddr returns the local network address.

src/pkg/net/tcpsock_posix.go

@@ -108,9 +108,7 @@ func (c *TCPConn) Close() error {
 	if !c.ok() {
 		return syscall.EINVAL
 	}
-	err := c.fd.Close()
+	return c.fd.Close()
-	c.fd = nil
-	return err
 }
 
 // CloseRead shuts down the reading side of the TCP connection.

src/pkg/net/udpsock_posix.go

@@ -88,9 +88,7 @@ func (c *UDPConn) Close() error {
 	if !c.ok() {
 		return syscall.EINVAL
 	}
-	err := c.fd.Close()
+	return c.fd.Close()
-	c.fd = nil
-	return err
 }
 
 // LocalAddr returns the local network address.

src/pkg/net/unixsock_posix.go

@@ -141,9 +141,7 @@ func (c *UnixConn) Close() error {
 	if !c.ok() {
 		return syscall.EINVAL
 	}
-	err := c.fd.Close()
+	return c.fd.Close()
-	c.fd = nil
-	return err
 }
 
 // LocalAddr returns the local network address, a *UnixAddr.