Merge pull request #7115 from kubo39/note-async-signal-safe-function

Add comment for using non async-signal-safe function
dlang · Jul 25, 2019 · 442ab4e · 442ab4e
2 parents e123a4d + 8dc81bb
commit 442ab4e
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/std/process.d b/std/process.d
@@ -509,6 +509,10 @@ private Pid spawnProcessImpl(scope const(char[])[] args,
 
             if (!(config & Config.inheritFDs))
             {
+                // NOTE: malloc() and getrlimit() are not on the POSIX async
+                // signal safe functions list, but practically this should not
+                // be a problem. Tha Java VM and CPython also use malloc() in
+                // its own implementation.
                 import core.stdc.stdlib : malloc;
                 import core.sys.posix.poll : pollfd, poll, POLLNVAL;
                 import core.sys.posix.sys.resource : rlimit, getrlimit, RLIMIT_NOFILE;