lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date:   Sat, 11 Jan 2020 19:23:30 +0100
From:   Christian Brauner <christian.brauner@...ntu.com>
To:     Linus Torvalds <torvalds@...ux-foundation.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: [GIT PULL] thread fixes v5.5-rc6

Hey Linus,

/* Summary */
This pull request contains a series of patches to fix CLONE_SETTLS when used
with clone3().
The clone3() syscall passes the tls argument through struct clone_args instead
of a register. This means, all architectures that do not implement
copy_thread_tls() but still support CLONE_SETTLS via copy_thread() expecting
the tls to be located in a register argument based on clone() are currently
unfortunately broken. Their tls value will be garbage.
The patch series I got sent fixes this on all architectures that currently
define __ARCH_WANT_SYS_CLONE3. It also adds a compile-time check to ensure that
any architecture that enables clone3() in the future is forced to also
implement copy_thread_tls().

My ultimate goal is to get rid of the copy_thread()/copy_thread_tls() split and
just have copy_thread_tls() at some point in the not too distant future (Maybe
even renaming copy_thread_tls() back to simply copy_thread() once the old
function is ripped from all arches). This is dependent now on all arches
supporting clone3(). While all relevant arches do that now there are still four
missing:
arch/ia64/kernel/syscalls/syscall.tbl:# 435 reserved for clone3
arch/m68k/kernel/syscalls/syscall.tbl:# 435 reserved for clone3
arch/sh/kernel/syscalls/syscall.tbl:# 435 reserved for clone3
arch/sparc/kernel/syscalls/syscall.tbl:# 435 reserved for clone3
Once they all implement clone3() we can get rid of ARCH_WANT_SYS_CLONE3 and
HAVE_COPY_THREAD_TLS.

This series also includes a minor fix for the arm64 uapi headers which caused
__NR_clone3 to be missing from the exported user headers.

Unfortunately the series came in a little late especially given that it touches
a range of architectures. Due to the holidays not all arch maintainers
responded in time probably due to their backlog. Will and Arnd have thankfully
acked the arm specific changes.
Given that the changes are straightforward and rather minimal combined with the
fact the that clone3() with CLONE_SETTLS is broken I decided to send them post
rc3 nonetheless. If you find this to be too late I can resend for the next
merge window but note that this means we need to backport the fixes to 5.5,
5.4, and 5.3 instead of just 5.4 and 5.3.

/* Testing */
All patches have seen exposure in linux-next and are based on v5.5-rc5.
The clone3 selftests passed. We will likely add more tests for CLONE_SETTLS in
the future but since this is highly architecture dependent we first need to
settle on how to test this easily (Some ideas are already discussed.).

/* Conflicts */
There was a minor merge conflict with a change to riscv's Kconfig but doing a
test merge with current mainline I was not able to reproduce it.

Please consider pulling these changes from the signed clone3-tls-v5.5-rc6 tag.

Thanks!
Christian

The following changes since commit c79f46a282390e0f5b306007bf7b11a46d529538:

  Linux 5.5-rc5 (2020-01-05 14:23:27 -0800)

are available in the Git repository at:

  git@...olite.kernel.org:pub/scm/linux/kernel/git/brauner/linux tags/clone3-tls-v5.5-rc6

for you to fetch changes up to 457677c70c7672a4586b0b8abc396cc1ecdd376d:

  um: Implement copy_thread_tls (2020-01-07 13:31:29 +0100)

----------------------------------------------------------------
clone3-tls-v5.5-rc6

----------------------------------------------------------------
Amanieu d'Antras (8):
      arm64: Move __ARCH_WANT_SYS_CLONE3 definition to uapi headers
      arm64: Implement copy_thread_tls
      arm: Implement copy_thread_tls
      parisc: Implement copy_thread_tls
      riscv: Implement copy_thread_tls
      xtensa: Implement copy_thread_tls
      clone3: ensure copy_thread_tls is implemented
      um: Implement copy_thread_tls

 arch/arm/Kconfig                     |  1 +
 arch/arm/kernel/process.c            |  6 +++---
 arch/arm64/Kconfig                   |  1 +
 arch/arm64/include/asm/unistd.h      |  1 -
 arch/arm64/include/uapi/asm/unistd.h |  1 +
 arch/arm64/kernel/process.c          | 10 +++++-----
 arch/parisc/Kconfig                  |  1 +
 arch/parisc/kernel/process.c         |  8 ++++----
 arch/riscv/Kconfig                   |  1 +
 arch/riscv/kernel/process.c          |  6 +++---
 arch/um/Kconfig                      |  1 +
 arch/um/include/asm/ptrace-generic.h |  2 +-
 arch/um/kernel/process.c             |  6 +++---
 arch/x86/um/tls_32.c                 |  6 ++----
 arch/x86/um/tls_64.c                 |  7 +++----
 arch/xtensa/Kconfig                  |  1 +
 arch/xtensa/kernel/process.c         |  8 ++++----
 kernel/fork.c                        | 10 ++++++++++
 18 files changed, 45 insertions(+), 32 deletions(-)

Powered by blists - more mailing lists