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

Enable ASAN checks on a CI native build #1010

Open
jaromil opened this issue Jan 20, 2025 · 0 comments
Open

Enable ASAN checks on a CI native build #1010

jaromil opened this issue Jan 20, 2025 · 0 comments
Labels

Comments

@jaromil
Copy link
Member

jaromil commented Jan 20, 2025

Following up on the merge of #999 the memcount internal mechanism of memory accounting in zenroom is removed in favor of ASAN reporting.

Therefore we should promote the ASAN tests to CI to monitor the memory handling.

To do this we need to create a check-asan test target that runs only tests that currently work under ASAN, which are the vast majority, excluding:

lua/native: events.lua

for a division by zero which is puzzling but does not affects us since we do not use co-routines

zencode_then (test/zencode/then.bats)

not ok 15 String dictionary with float and time
(in test file /home/jrml/devel/zenroom/test/zencode/then.bats, line 265)

src/zen_time.c:174:11: runtime error: 1.23457e+10 is outside the range of representable values of type 'int' in detect_time_value src/zen_time.c:174

api_hash (all API tests)

`LD_LIBRARY_PATH=$R ./hash_init sha512 > ctx_sha512' failed with status 134
ASan runtime does not come first in initial library list; you should either link runtime to your application or manually preload it with LD_PRELOAD.

zencode w3c

zencode_w3c TIMEOUT 59.81s killed by signal 15 SIGTERM

@jaromil jaromil added the CI/CD label Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant