added enable_asan option

.github/WORKFLOW-PARAMETERS.md

@@ -12,3 +12,6 @@ Setting this parameter will cause the test program to be run with `valgrind`.
 ```shell
 git commit -m "commit message --filter=[core][unit] --valgrind"
 ```
+
+## --asan
+Setting this parameter will cause the test program to be run with `ASAN`.

.github/workflows/core.yml

@@ -43,6 +43,11 @@ jobs:
  sudo apt install -y valgrind
  cd core-tests && SWOOLE_VALGRIND=1 ./run.sh
 
+ - name: make test with asan
+ if: "contains(github.event.head_commit.message, '--asan')"
+ run: |
+ cd core-tests && SWOOLE_ENABLE_ASAN=1 ./run.sh
+
  - name: make test
  run:
  cd core-tests && ./run.sh

CMakeLists.txt

@@ -100,6 +100,11 @@ if (DEFINED enable_trace_log)
  add_definitions(-DSW_LOG_TRACE_OPEN)
 endif()
 
+if (DEFINED enable_asan)
+ set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fsanitize=address -fno-omit-frame-pointer")
+ set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fsanitize=address -fno-omit-frame-pointer")
+endif()
+
 execute_process(COMMAND php-config --includes OUTPUT_VARIABLE PHP_INCLUDES OUTPUT_STRIP_TRAILING_WHITESPACE)
 execute_process(COMMAND php-config --extension-dir OUTPUT_VARIABLE PHP_EXTENSION_DIR OUTPUT_STRIP_TRAILING_WHITESPACE)
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${PHP_INCLUDES}")

core-tests/CMakeLists.txt

@@ -61,6 +61,10 @@ if (DEFINED enable_asan)
  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fsanitize=address -fno-omit-frame-pointer")
 endif()
 
+if (DEFINED enable_thread)
+ add_definitions(-DSW_THREAD)
+endif()
+
 # should execute before the add_executable command
 link_directories(${core_tests_link_directories})
 

core-tests/run.sh

@@ -1,5 +1,9 @@
 #!/bin/bash
-cmake . -DSW_THREAD=1
+if [ "${SWOOLE_ENABLE_ASAN}" = 1 ]; then
+ cmake . -D enable_thread=1 -D enable_asan=1
+else
+ cmake . -D enable_thread=1
+fi
 make -j8
 ipcs -q