From c176680443eb81161c888ecf023709d19745ec85 Mon Sep 17 00:00:00 2001 From: Daniel Bertalan Date: Thu, 15 Jul 2021 12:05:08 +0200 Subject: [PATCH] Kernel: Tell the compiler about `operator new`'s alignment By default, the compiler will assume that `operator new` returns pointers that are aligned correctly for every built-in type. This is not the case in the kernel on x64, since the assumed alignment is 16 (because of long double), but the kmalloc blocks are only `alignas(void*)`. --- Kernel/CMakeLists.txt | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Kernel/CMakeLists.txt b/Kernel/CMakeLists.txt index 8b2ace0027..0e59bd169b 100644 --- a/Kernel/CMakeLists.txt +++ b/Kernel/CMakeLists.txt @@ -356,8 +356,10 @@ endif() if ("${SERENITY_ARCH}" STREQUAL "x86_64") set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -mcmodel=large -fno-pic -mno-red-zone") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -faligned-new=8") else() set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -pie -fPIE") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -faligned-new=4") endif() # Kernel Undefined Behavior Sanitizer (KUBSAN)