# HG changeset patch
# User Yuya Nishihara <yuya@tcha.org>
# Date 2016-08-05 12:21:33
# Node ID acf27be56d26ab8055a3f6a52232c590038c9fdb
# Parent  2dd8c225e94c19d03713458bbeacc71c30347c51

chg: just take it as EOF if recv() returns 0

hgc->sockfd is a blocking stream socket. recv() should never return 0 other
than EOF.

See 4fc4b8cc9957 for the original problem.

diff --git a/contrib/chg/hgclient.c b/contrib/chg/hgclient.c
--- a/contrib/chg/hgclient.c
+++ b/contrib/chg/hgclient.c
@@ -126,15 +126,10 @@ static void readchannel(hgclient_t *hgc)
 		return;  /* assumes input request */
 
 	size_t cursize = 0;
-	int emptycount = 0;
 	while (cursize < hgc->ctx.datasize) {
 		rsize = recv(hgc->sockfd, hgc->ctx.data + cursize,
 			     hgc->ctx.datasize - cursize, 0);
-		/* rsize == 0 normally indicates EOF, while it's also a valid
-		 * packet size for unix socket. treat it as EOF and abort if
-		 * we get many empty responses in a row. */
-		emptycount = (rsize == 0 ? emptycount + 1 : 0);
-		if (rsize < 0 || emptycount > 20)
+		if (rsize < 1)
 			abortmsg("failed to read data block");
 		cursize += rsize;
 	}