Skip to content

Commit

Permalink
Workaround leak sanitizer false positive
Browse files Browse the repository at this point in the history
  • Loading branch information
nsavoire committed Apr 2, 2024
1 parent 7fb5c54 commit f44f806
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -207,14 +207,12 @@ if(USE_LOADER)
endif()

set(DD_PROFILING_SOURCES
# cmake-format: sortable
src/daemonize.cc
src/ddprof_cmdline.cc
src/ddres_list.cc
src/ipc.cc
src/lib/address_bitset.cc
src/lib/allocation_tracker.cc
src/lib/dd_profiling.cc
src/lib/elfutils.cc
src/lib/lib_embedded_data.c
src/lib/pthread_fixes.cc
Expand All @@ -234,7 +232,10 @@ set(DD_PROFILING_SOURCES
src/sys_utils.cc
src/tracepoint_config.cc
src/tsc_clock.cc
src/user_override.cc)
src/user_override.cc
# Intentionally put dd_profiling.cc last to avoid leak sanitizer failure complaining that memory
# in LoggerContext::name is not freed
src/lib/dd_profiling.cc)

if(BUILD_UNIVERSAL_DDPROF)
# Compiling on different libc, we need to ensure some symbols are available everywhere
Expand Down

0 comments on commit f44f806

Please sign in to comment.