Skip to content

Commit

Permalink
fix(libsinsp): no_sanitize(undefined)
Browse files Browse the repository at this point in the history
Signed-off-by: Luca Guerra <[email protected]>
  • Loading branch information
LucaGuerra committed Mar 5, 2024
1 parent 79bf5ff commit c7e9d83
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
2 changes: 2 additions & 0 deletions .github/sanitizers-ignore.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
# some gRPC versions cannot be run with UBSan
fun:*grpc*
4 changes: 2 additions & 2 deletions .github/workflows/e2e_ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ jobs:
- name: Run e2e tests with ${{ matrix.driver.name }} 🏎️
if: matrix.arch == 'amd64'
env:
UBSAN_OPTIONS: print_stacktrace=1
UBSAN_OPTIONS: print_stacktrace=1;suppressions=.github/sanitizers-ignore.txt
run: |
cd build/test/libsinsp_e2e/
sudo -E ./libsinsp_e2e_tests ${{ matrix.driver.option }}
Expand All @@ -105,7 +105,7 @@ jobs:
- name: Run e2e tests with ${{ matrix.driver.name }} 🏎️
if: matrix.arch == 'arm64'
env:
UBSAN_OPTIONS: print_stacktrace=1
UBSAN_OPTIONS: print_stacktrace=1;suppressions=.github/sanitizers-ignore.txt
run: |
cd build/test/libsinsp_e2e/
sudo -E ./libsinsp_e2e_tests ${{ matrix.driver.option }} --gtest_filter=-sys_call_test.quotactl_ok
2 changes: 1 addition & 1 deletion userspace/libsinsp/cri.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ namespace libsinsp
namespace cri
{

template<typename api>
template<typename api>
inline cri_interface<api>::cri_interface(const std::string &cri_path)
{
std::shared_ptr<grpc::Channel> channel = libsinsp::grpc_channel_registry::get_channel("unix://" + cri_path);
Expand Down

0 comments on commit c7e9d83

Please sign in to comment.