Fix sanitizer CI for the C++ KMIP client

The backend failed to load pg_tde.so under sanitizers with "undefined
symbol: __ubsan_vptr_type_cache", so every test that starts a server
errored. That symbol is provided only by the C++-only UBSan runtime
(ubsan_standalone_cxx), which is not linked into the C postgres backend
that dlopens the module. Disable the vptr check for the C++ objects and
build the C++ sources/links with the sanitizer flags (CXX=clang++) so a
single clang ASan/UBSan runtime is used throughout.

Also drop the non-existent llvm-symbolizer-14 path so ASan locates a
working symbolizer, and ignore geninfo version mismatches in coverage.

Author: dutow

.github/workflows/coverage.yml

@@ -50,8 +50,8 @@ jobs:
 
       - name: Process coverage
         run: |
-          geninfo -o coverage.info --rc lcov_branch_coverage=1 -i .
-          geninfo -o coverage.info --rc lcov_branch_coverage=1 .
+          geninfo -o coverage.info --rc lcov_branch_coverage=1 --ignore-errors mismatch -i .
+          geninfo -o coverage.info --rc lcov_branch_coverage=1 --ignore-errors mismatch .
         working-directory: build
 
       - name: Upload coverage data to codecov.io

.github/workflows/sanitizers.yml

@@ -12,11 +12,11 @@ on:
 env:
   pg_version: 18
   CC: clang
+  CXX: clang++
   LD: clang
   UBSAN_OPTIONS: log_path=${{ github.workspace }}/sanitize.log print_suppressions=0 print_stacktrace=1 print_summary=1 halt_on_error=1
   ASAN_OPTIONS: log_path=${{ github.workspace }}/sanitize.log print_suppressions=0 abort_on_error=1
   LSAN_OPTIONS: log_path=${{ github.workspace }}/sanitize.log print_suppressions=0 suppressions=${{ github.workspace }}/src/ci_scripts/suppressions/lsan.supp
-  ASAN_SYMBOLIZER_PATH: /usr/bin/llvm-symbolizer-14
   # Avoid failures on slow recovery
   PGCTLTIMEOUT: 120
   PG_TEST_TIMEOUT_DEFAULT: 300

ci_scripts/build.sh

@@ -29,7 +29,10 @@ case "$1" in
         BUILD_TYPE=debug
         ARGS+=" -Dc_args=['-fsanitize=address','-fsanitize=undefined','-fno-omit-frame-pointer','-fno-inline-functions']"
         ARGS+=" -Dc_link_args=['-fsanitize=address','-fsanitize=undefined']"
-        ARGS+=" -Dcpp_args=['-fsanitize=address','-fsanitize=undefined','-fno-omit-frame-pointer','-fno-inline-functions']"
+        # vptr's UBSan runtime lives in the C++-only ubsan_standalone_cxx library,
+        # which the C postgres backend that dlopens pg_tde.so does not link, so it
+        # is disabled here to avoid an undefined __ubsan_vptr_type_cache at load.
+        ARGS+=" -Dcpp_args=['-fsanitize=address','-fsanitize=undefined','-fno-sanitize=vptr','-fno-omit-frame-pointer','-fno-inline-functions']"
         ARGS+=" -Dcpp_link_args=['-fsanitize=address','-fsanitize=undefined']"
         ;;