We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

[patch] patch enable AddressSanitizer if supported

0001-cmake-enable-AddressSanitizer-in-Debug-builds-if-sup.patch (text/plain), 2.66 KB, created by Christian Rauch on 2021-02-02 00:45:25 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Christian Rauch

Created: 2021-02-02 00:45:25 UTC

Size: 2.66 KB

patch

obsolete

>From 5303b0db9f68bfd5b2e75f236b375f6655a5e54a Mon Sep 17 00:00:00 2001
>From: Christian Rauch <Rauch.Christian@gmx.de>
>Date: Sun, 14 Jun 2020 01:39:05 +0100
>Subject: [PATCH] cmake: enable AddressSanitizer in Debug builds if supported
>
>---
> CMakeLists.txt | 51 ++++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 51 insertions(+)
>
>diff --git a/CMakeLists.txt b/CMakeLists.txt
>index 40ddc0d03..4e20eefd2 100644
>--- a/CMakeLists.txt
>+++ b/CMakeLists.txt
>@@ -34,6 +34,7 @@ include(CheckSymbolExists)
> include(CheckCSourceCompiles)
> include(CheckCSourceRuns)
> include(CheckCCompilerFlag)
>+include(CheckCXXCompilerFlag)
> include(CheckTypeSize)
> include(CheckStructHasMember)
> include(CMakeDependentOption)
>@@ -2323,6 +2324,56 @@ if(SDL_STATIC)
>   endif()
> endif()
> 
>+macro(asan_check_add_debug_flag ASAN_FLAG)
>+    set(FLAG "-fsanitize=${ASAN_FLAG}")
>+
>+    check_c_compiler_flag(${FLAG} HAS_C_FLAG_${ASAN_FLAG})
>+    if (HAS_C_FLAG_${ASAN_FLAG})
>+        set(CMAKE_C_FLAGS_DEBUG "${CMAKE_C_FLAGS_DEBUG} ${FLAG}")
>+    endif()
>+
>+    check_cxx_compiler_flag("${FLAG}" HAS_CXX_${ASAN_FLAG})
>+    if (HAS_CXX_${ASAN_FLAG})
>+        set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} ${FLAG}")
>+    endif()
>+endmacro()
>+
>+macro(asan_check_add_debug_flag2 ASAN_FLAG)
>+    # for some sanitize flags we have to manipulate the CMAKE_REQUIRED_LIBRARIES:
>+    # http://cmake.3232098.n2.nabble.com/CHECK-CXX-COMPILER-FLAG-doesn-t-give-correct-result-for-fsanitize-address-tp7600216p7600217.html
>+
>+    set(FLAG "-fsanitize=${ASAN_FLAG}")
>+
>+    set (STORED_REQLIBS ${CMAKE_REQUIRED_LIBRARIES})
>+    set (CMAKE_REQUIRED_LIBRARIES "${FLAG};asan")
>+    check_c_compiler_flag (${FLAG} HAS_C_FLAG_${ASAN_FLAG})
>+    check_cxx_compiler_flag (${FLAG} HAS_CXX_FLAG_${ASAN_FLAG})
>+    set (CMAKE_REQUIRED_LIBRARIES ${STORED_REQLIBS})
>+
>+    if (HAS_C_FLAG_${ASAN_FLAG})
>+        set(CMAKE_C_FLAGS_DEBUG "${CMAKE_C_FLAGS_DEBUG} ${FLAG}")
>+    endif()
>+
>+    if (HAS_CXX_${ASAN_FLAG})
>+        set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} ${FLAG}")
>+    endif()
>+endmacro()
>+
>+# enable AddressSanitizer if supported
>+asan_check_add_debug_flag2("address")
>+asan_check_add_debug_flag("bool")
>+asan_check_add_debug_flag("bounds")
>+asan_check_add_debug_flag("enum")
>+asan_check_add_debug_flag("float-cast-overflow")
>+asan_check_add_debug_flag("float-divide-by-zero")
>+asan_check_add_debug_flag("nonnull-attribute")
>+asan_check_add_debug_flag("returns-nonnull-attribute")
>+asan_check_add_debug_flag("signed-integer-overflow")
>+asan_check_add_debug_flag("undefined")
>+asan_check_add_debug_flag("vla-bound")
>+asan_check_add_debug_flag("leak")
>+asan_check_add_debug_flag("object-size")
>+
> ##### Tests #####
> 
> if(SDL_TEST)
>-- 
>2.25.1
>

Actions: View | Diff

Attachments on bug 5521: 4741 | 4748

We are currently migrating Bugzilla to GitHub issues.Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.