Thread (1 message) 1 message, 1 author, 2016-06-15

Re: GIT_OBJECT_DIRECTORY

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:42:23
Subsystem: the rest · Maintainer: Linus Torvalds

Possibly related (same subject, not in this thread)

"H. Peter Anvin" [off-list ref] writes:
Jörn Engel wrote:
quoted
Excellent!  I have a faint memory of hpa recently saying that the git
daemon would be too resource-hungry.  One of the cases where being
wrong is a Good Thing.
Well, we ended up making some tweaks to the git daemon, and it hasn't
been a problem since.
Ah, I am glad the daemon expert was listening...  Do you have
comments on recent patch from Serge E. Hallyn?  It looks OK to
me, but that standalone daemon part is not something I run
myself, so...

-- >8 --
[PATCH] socksetup: don't return on set_reuse_addr() error

The set_reuse_addr() error case was the only error case in
socklist() where we returned rather than continued.  Not sure
why.  Either we must free the socklist, or continue.  This patch
continues on error.

Signed-off-by: Serge E. Hallyn <redacted>
Signed-off-by: Junio C Hamano <redacted>

---

 daemon.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

0032d548db56eac9ea09b4ba05843365f6325b85
diff --git a/daemon.c b/daemon.c
index a1ccda3..776749e 100644
--- a/daemon.c
+++ b/daemon.c
@@ -535,7 +535,7 @@ #endif
 
 		if (set_reuse_addr(sockfd)) {
 			close(sockfd);
-			return 0;	/* not fatal */
+			continue;
 		}
 
 		if (bind(sockfd, ai->ai_addr, ai->ai_addrlen) < 0) {
-- 
1.3.0.rc4.g5247-dirty
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help