[tor-commits] [tor/release-0.2.4] Fix another case of bug 8206; patch from flupzor

arma at torproject.org arma at torproject.org
Thu Apr 11 05:29:51 UTC 2013


commit 0b827cbcb11abece686d402170bd58724a77e365
Author: Nick Mathewson <nickm at torproject.org>
Date:   Mon Mar 18 15:44:23 2013 -0400

    Fix another case of bug 8206; patch from flupzor
---
 src/common/compat.c |   20 ++++++++++++++++----
 1 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/src/common/compat.c b/src/common/compat.c
index 25df9a9..4fa9fee 100644
--- a/src/common/compat.c
+++ b/src/common/compat.c
@@ -1172,10 +1172,22 @@ tor_socketpair(int family, int type, int protocol, tor_socket_t fd[2])
     return -errno;
 
 #if defined(FD_CLOEXEC)
-  if (SOCKET_OK(fd[0]))
-    fcntl(fd[0], F_SETFD, FD_CLOEXEC);
-  if (SOCKET_OK(fd[1]))
-    fcntl(fd[1], F_SETFD, FD_CLOEXEC);
+  if (SOCKET_OK(fd[0])) {
+    r = fcntl(fd[0], F_SETFD, FD_CLOEXEC);
+    if (r == -1) {
+      close(fd[0]);
+      close(fd[1]);
+      return -errno;
+    }
+  }
+  if (SOCKET_OK(fd[1])) {
+    r = fcntl(fd[1], F_SETFD, FD_CLOEXEC);
+    if (r == -1) {
+      close(fd[0]);
+      close(fd[1]);
+      return -errno;
+    }
+  }
 #endif
   goto sockets_ok; /* So that sockets_ok will not be unused. */
 





More information about the tor-commits mailing list