]> git.cameronkatri.com Git - pw-darwin.git/commitdiff
Remove a death threat from the FreeBSD sources
authorConrad Meyer <cem@FreeBSD.org>
Thu, 1 Dec 2016 02:21:36 +0000 (02:21 +0000)
committerConrad Meyer <cem@FreeBSD.org>
Thu, 1 Dec 2016 02:21:36 +0000 (02:21 +0000)
Reported by: koobs@, araujo@, linimon@, bjk@, emaste@, jhb@, ngie@, cem@
Maintainer timeout: des@

libutil/flopen.c

index bc4119dcb1cc15d6e1791816c9b85a0d87673092..89297290796e32dd7d0381e4172d1eb19c487d31 100644 (file)
@@ -40,10 +40,10 @@ __FBSDID("$FreeBSD$");
 /*
  * Reliably open and lock a file.
  *
- * DO NOT, UNDER PAIN OF DEATH, modify this code without first reading the
- * revision history and discussing your changes with <des@freebsd.org>.
- * Don't be fooled by the code's apparent simplicity; there would be no
- * need for this function if it was as easy to get right as you think.
+ * Please do not modify this code without first reading the revision history
+ * and discussing your changes with <des@freebsd.org>.  Don't be fooled by the
+ * code's apparent simplicity; there would be no need for this function if it
+ * was easy to get right.
  */
 int
 flopen(const char *path, int flags, ...)
@@ -108,7 +108,11 @@ flopen(const char *path, int flags, ...)
                        errno = serrno;
                        return (-1);
                }
-#ifdef DONT_EVEN_THINK_ABOUT_IT
+               /*
+                * The following change is provided as a specific example to
+                * avoid.
+                */
+#if 0
                if (fcntl(fd, F_SETFD, FD_CLOEXEC) != 0) {
                        serrno = errno;
                        (void)close(fd);