Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tarantool: work around LeakSanitizer internal errors #12321

Merged

Conversation

ligurio
Copy link
Contributor

@ligurio ligurio commented Aug 9, 2024

With enabled LeakSanitizer on AArch64 ICU build fails due to false positive warnings from Leak Sanitizer (part of config.log):

configure:3209: checking whether we are cross compiling configure:3217: clang -o conftest -O1 -fno-omit-frame-pointer -gline-tables-only -DFUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION -fsanitize=address -fsanitize-address-use-after-scope -fsanitize=fuzzer-no-link  -lpthread conftest.c  >&5 configure:3221: $? = 0
configure:3228: ./conftest
==666==LeakSanitizer has encountered a fatal error. ==666==HINT: For debugging, try setting environment variable LSAN_OPTIONS=verbosity=1:log_threads=1 ==666==HINT: LeakSanitizer does not work under ptrace (strace, gdb, etc) configure:3232: $? = 1
configure:3239: error: in `/src/tarantool/build/icu': configure:3241: error: cannot run C compiled programs.

The patch disable LeakSanitizer on AArch64 and increase verbosity.

Related to #11798
Related to google/sanitizers#723

With enabled LeakSanitizer on AArch64 ICU build fails due to
false positive warnings from Leak Sanitizer (part of config.log):

```
configure:3209: checking whether we are cross compiling
configure:3217: clang -o conftest -O1 -fno-omit-frame-pointer -gline-tables-only -DFUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION -fsanitize=address -fsanitize-address-use-after-scope -fsanitize=fuzzer-no-link  -lpthread conftest.c  >&5
configure:3221: $? = 0
configure:3228: ./conftest
==666==LeakSanitizer has encountered a fatal error.
==666==HINT: For debugging, try setting environment variable LSAN_OPTIONS=verbosity=1:log_threads=1
==666==HINT: LeakSanitizer does not work under ptrace (strace, gdb, etc)
configure:3232: $? = 1
configure:3239: error: in `/src/tarantool/build/icu':
configure:3241: error: cannot run C compiled programs.
```

The patch disable LeakSanitizer on AArch64 and increase verbosity.

Related to google#11798
Related to google/sanitizers#723
@ligurio ligurio force-pushed the ligurio/fix-aarch64-tarantool branch from 77c07d4 to 9a21260 Compare August 9, 2024 07:26
Copy link

github-actions bot commented Aug 9, 2024

ligurio is either the primary contact or is in the CCs list of projects/tarantool.
ligurio has previously contributed to projects/tarantool. The previous PR was #12314

@ligurio
Copy link
Contributor Author

ligurio commented Aug 14, 2024

@DavidKorczynski could you please review?

@DavidKorczynski DavidKorczynski merged commit 758349e into google:master Aug 14, 2024
15 checks passed
@ligurio ligurio deleted the ligurio/fix-aarch64-tarantool branch August 14, 2024 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants