diff options
| author | agl@chromium.org <agl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-10 19:00:58 +0000 |
|---|---|---|
| committer | agl@chromium.org <agl@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-10 19:00:58 +0000 |
| commit | cd3cf12588560bf1c1ffc780ce7fdeae044b100e (patch) | |
| tree | 8ab79805aa9e50db3a2eae36e324af2f92bc13d4 /base/process_util.h | |
| parent | 2fc4c21d0b6cebaf8ebc101cc3fe20ef38407373 (diff) | |
| download | chromium_src-cd3cf12588560bf1c1ffc780ce7fdeae044b100e.zip chromium_src-cd3cf12588560bf1c1ffc780ce7fdeae044b100e.tar.gz chromium_src-cd3cf12588560bf1c1ffc780ce7fdeae044b100e.tar.bz2 | |
POSIX: don't allocate memory after forking.
Previously we would allocate memory in the child process. However, the
allocation might have happened while the malloc lock was held,
resulting in a deadlock.
This patch removes allocation from the child but probably makes Mac's
startup time slower until a Mac person can implement
dir_reader_posix.h.
TEST=Unittest for new code
BUG=36678
http://codereview.chromium.org/672003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41181 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/process_util.h')
| -rw-r--r-- | base/process_util.h | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/base/process_util.h b/base/process_util.h index cf4bd01..0c8c474 100644 --- a/base/process_util.h +++ b/base/process_util.h @@ -111,12 +111,6 @@ bool AdjustOOMScore(ProcessId process, int score); #endif #if defined(OS_POSIX) -// Sets all file descriptors to close on exec except for stdin, stdout -// and stderr. -// TODO(agl): remove this function -// WARNING: do not use. It's inherently race-prone in the face of -// multi-threading. -void SetAllFDsToCloseOnExec(); // Close all file descriptors, expect those which are a destination in the // given multimap. Only call this function in a child process where you know // that there aren't any other threads. |