summaryrefslogtreecommitdiff
path: root/builtin-archive.c
diff options
context:
space:
mode:
authorJunio C Hamano <junkio@cox.net>2007-01-21 17:10:51 -0800
committerJunio C Hamano <junkio@cox.net>2007-01-21 21:51:01 -0800
commitec587fde0a76780931c7ac32474c8c000aa45134 (patch)
tree8b1e608dd2f2641adbfe06cc4c0047ceb88c9d53 /builtin-archive.c
parent026aa93818a536c819a94aae5bbefe1b6251fe0e (diff)
downloadgit-ec587fde0a76780931c7ac32474c8c000aa45134.tar.gz
Make sure git_connect() always give two file descriptors.
Earlier, git_connect() returned the same fd twice or two separate fds, depending on the way the connection was made (when we are talking to the other end over a single socket, we used the same fd twice, and when our end is connected to a pipepair we used two). This forced callers who do close() and dup() to really care which was which, and most of the existing callers got this wrong, although without much visible ill effect. Many were closing the same fd twice when we are talking over a single socket, and one was leaking a fd. This fixes it to uniformly use two separate fds, so if somebody wants to close only reader side can just do close() on it without worrying about it accidentally also closing the writer side or vice versa. Signed-off-by: Linus Torvalds <torvalds@osdl.org> Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'builtin-archive.c')
-rw-r--r--builtin-archive.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/builtin-archive.c b/builtin-archive.c
index 32737d3162..f613ac2516 100644
--- a/builtin-archive.c
+++ b/builtin-archive.c
@@ -74,6 +74,7 @@ static int run_remote_archiver(const char *remote, int argc,
/* Now, start reading from fd[0] and spit it out to stdout */
rv = recv_sideband("archive", fd[0], 1, 2);
close(fd[0]);
+ close(fd[1]);
rv |= finish_connect(pid);
return !!rv;