diff options
author | Corinna Vinschen <corinna-cygwin@cygwin.com> | 2021-10-26 19:03:28 +0300 |
---|---|---|
committer | Corinna Vinschen <corinna@vinschen.de> | 2021-10-28 23:35:17 +0300 |
commit | e528cfe919d757ba9bc07db1798cb17560bba457 (patch) | |
tree | 95564384f7627a58dd29c78bf26e138e7daef85b /winsup/cygwin/cygmalloc.h | |
parent | 462ca33745cfa45302eeeb1d6e6bd82cc64c4027 (diff) |
Cygwin: split malloc_initcygwin-3_3_1-release
Cygwin 3.3 only: Replace SRWLOCK usage for malloc synchronization with
the first incarnation of the patch splitting malloc_init into two parts.
The SRWLOCK usage requires TryAcquireSRWLockExclusive, which isn't
available on Vista / Server 2008, unfortunately.
Per https://cygwin.com/pipermail/cygwin-developers/2021-October/012429.html,
we may encounter a crash when starting multiple threads during process
startup (here: fhandler_fifo::fixup_after_{fork,exec}) which in turn
allocate memory via malloc.
The problem is concurrent usage of malloc before the malloc muto has
been initialized.
To fix this issue, split malloc_init into malloc_init_0, called from
dll_crt0_0, and malloc_init_1, called from dll_crt_0_1. malloc_init_0
just initializes the muto, malloc_init_1 checks for user space provided
malloc.
Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
Diffstat (limited to 'winsup/cygwin/cygmalloc.h')
-rw-r--r-- | winsup/cygwin/cygmalloc.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/winsup/cygwin/cygmalloc.h b/winsup/cygwin/cygmalloc.h index 84bad824c..784b6c1a7 100644 --- a/winsup/cygwin/cygmalloc.h +++ b/winsup/cygwin/cygmalloc.h @@ -39,6 +39,8 @@ void *mmap64 (void *, size_t, int, int, int, off_t); # define __malloc_lock() mallock.acquire () # define __malloc_unlock() mallock.release () extern muto mallock; +inline void malloc_init_0 () { mallock.init ("mallock"); } +extern void malloc_init_1 (); #endif |