Revert: client.go: HybridVSockDialer: Close dup fd after receive packet

This reverts commit 33f3208. Revert changes as bring some instability to CI. After found the root cause it will be merged again. Fixes: kata-containers#716 Signed-off-by: Jose Carlos Venegas Munoz <[email protected]>
jcvenegas · Jan 24, 2020 · 54eb918 · 54eb918
1 parent 7c2d8ab
commit 54eb918
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/protocols/client/client.go b/protocols/client/client.go
@@ -419,9 +419,7 @@ func HybridVSockDialer(sock string, timeout time.Duration) (net.Conn, error) {
 				return nil, err
 			}
 			eot := make([]byte, 1)
-			n, _, err := unix.Recvfrom(int(file.Fd()), eot, syscall.MSG_PEEK)
-			file.Close()
-			if err != nil || n == 0 {
+			if n, _, err := unix.Recvfrom(int(file.Fd()), eot, syscall.MSG_PEEK); err != nil || n == 0 {
 				conn.Close()
 				if err == nil {
 					err = io.EOF