From: ian <ian>
Date: Sun, 15 Jun 2003 18:17:24 +0000 (+0000)
Subject: @@ -10,6 +10,8 @@
X-Git-Tag: debian_version_1_0_1_99_0~8
X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ian/git?a=commitdiff_plain;h=ccef06106c0b5e60fcbb9a636fd784638e9fc902;p=userv.git

@@ -10,6 +10,8 @@
+  * Use fcntl F_{GET,SET}FD with respect for as-yet-uninvented fd flags.
+    (small patch from Ben Harris.)
---

diff --git a/debian/changelog b/debian/changelog
index 5bf6e6a..248f460 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -10,6 +10,8 @@ userv (1.0.2) unstable; urgency=low
   * We do ship m4 and flex output now, so say so.
   * install-sh updated to that from autoconf 2.53.
   * Look for gmd5sum.  (Thanks to Anton Altaparmakov for the report.)
+  * Use fcntl F_{GET,SET}FD with respect for as-yet-uninvented fd flags.
+    (small patch from Ben Harris.)
 
   Debian:
   * Priority changed to optional as per override file.
diff --git a/servexec.c b/servexec.c
index a71b64f..8c3494a 100644
--- a/servexec.c
+++ b/servexec.c
@@ -224,7 +224,7 @@ void execservice(const int synchsocket[], int clientfd) {
     "-",
     0
   };
-  int fd, realfd, holdfd, newfd, r, envvarbufsize=0, targ, nargs, i, l;
+  int fd, realfd, holdfd, newfd, r, envvarbufsize=0, targ, nargs, i, l, fdflags;
   char *envvarbuf=0;
   const char **args, *const *cpp;
   char *const *pp;
@@ -286,7 +286,8 @@ void execservice(const int synchsocket[], int clientfd) {
     }
     if (dup2(fdarray[fd].realfd,fd)<0) serv_syscallfail("dup2 set up fd");
     if (close(fdarray[fd].realfd)) serv_syscallfail("close old fd");
-    if (fcntl(fd,F_SETFD,0)<0) serv_syscallfail("set no-close-on-exec on fd");
+    fdflags= fcntl(fd,F_GETFD); if (fdflags<0) serv_syscallfail("get fd flags");
+    if (fcntl(fd,F_SETFD,fdflags&~FD_CLOEXEC)==-1) serv_syscallfail("set no-close-on-exec on fd");
     fdarray[fd].realfd= fd;
   }