14 lines
631 B
Diff
14 lines
631 B
Diff
this optimisation of CLONE_SETTLS is not valid used like this, and future musl
|
|
clone(3) will EINVAL on this use
|
|
--
|
|
--- a/sandbox/linux/services/credentials.cc
|
|
+++ b/sandbox/linux/services/credentials.cc
|
|
@@ -89,7 +89,7 @@
|
|
|
|
int clone_flags = CLONE_FS | LINUX_SIGCHLD;
|
|
void* tls = nullptr;
|
|
-#if (defined(ARCH_CPU_X86_64) || defined(ARCH_CPU_ARM_FAMILY)) && \
|
|
+#if 0 && (defined(ARCH_CPU_X86_64) || defined(ARCH_CPU_ARM_FAMILY)) && \
|
|
!defined(MEMORY_SANITIZER)
|
|
// Use CLONE_VM | CLONE_VFORK as an optimization to avoid copying page tables.
|
|
// Since clone writes to the new child's TLS before returning, we must set a
|