Add an explicit check for broken libunwind builds

Libunwind as packaged in MacOS cannot actually unwind code which has
pointer authentication on, because _LIBUNWIND_IS_NATIVE_ONLY is not
defined. Add a check for this, and prefer building with working
unwinding over pointer authentication if we must make a choice.
This commit is contained in:
KJ Tsanaktsidis 2024-09-06 16:06:16 +10:00 коммит произвёл Peter Zhu
Родитель 2865148a5a
Коммит 731805ddde
1 изменённых файлов: 33 добавлений и 13 удалений

Просмотреть файл

@ -839,7 +839,26 @@ AS_IF([test "$GCC" = yes], [
])
# aarch64 branch protection
AS_CASE(["$target_cpu"], [aarch64], [
AS_CASE(["$target_cpu"], [aarch64|arm64], [
# LLVM libunwind is not actually capable of unwinding code compiled with pointer
# authentication unless it's built without LIBUNWIND_ENABLE_CROSS_UNWINDING (see
# https://github.com/llvm/llvm-project/blob/8e35c86977ce5529a9387657321ac9fefcdae5b5/libunwind/src/DwarfInstructions.hpp#L294)
# It seems that macOS ships LLVM compiled this way.
# Detect this and disable automatic insertion of pac-ret flags in that case, since we assume
# that reliable backtraces are more important than hardening flags.
AC_MSG_CHECKING([for a broken LLVM libunwind that cannot unwind code with RA signing])
AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[
@%:@include <libunwind.h>
int foo = UNW_ECROSSRASIGNING;
]])],
# If compilation succeeds, that means we a) had libunwind, and b) it was NOT native only
[rb_cv_libunwind_broken_ra_signing=yes],
# if compilation fails, that means we either a) do not have libunwind, or b) have it in
# native only mode (which is good!)
[rb_cv_libunwind_broken_ra_signing=no]
)
AC_MSG_RESULT(["$rb_cv_libunwind_broken_ra_signing"])
AS_IF([test "x$rb_cv_libunwind_broken_ra_signing" = "xno"], [
AS_FOR(option, opt, [-mbranch-protection=pac-ret -msign-return-address=all], [
# Try these flags in the _prepended_ position - i.e. we want to try building a program
# with CFLAGS="-mbranch-protection=pac-ret $CFLAGS". If the builder has provided different
@ -855,6 +874,7 @@ AS_IF([test "$GCC" = yes], [
])
])
])
])
AS_CASE("${compress_debug_sections:-zlib}",
[none|no], [], [