Revision: 76849
          http://sourceforge.net/p/brlcad/code/76849
Author:   starseeker
Date:     2020-08-19 14:54:56 +0000 (Wed, 19 Aug 2020)
Log Message:
-----------
Stash a note - not sure where the number 20 came from here...

Modified Paths:
--------------
    brlcad/trunk/src/libbu/process.c

Modified: brlcad/trunk/src/libbu/process.c
===================================================================
--- brlcad/trunk/src/libbu/process.c    2020-08-19 14:18:47 UTC (rev 76848)
+++ brlcad/trunk/src/libbu/process.c    2020-08-19 14:54:56 UTC (rev 76849)
@@ -332,6 +332,8 @@
        (void)close(pipe_err[0]);
        (void)close(pipe_err[1]);
 
+       // TODO - should we be doing this for more than 20? See
+       // 
https://docs.fedoraproject.org/en-US/Fedora_Security_Team/1/html/Defensive_Coding/sect-Defensive_Coding-Tasks-Descriptors-Child_Processes.html
        for (int i = 3; i < 20; i++) {
            (void)close(i);
        }

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.



_______________________________________________
BRL-CAD Source Commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-commits

Reply via email to