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

Memory leaks found while running tests. #1512

Open
hovhannes-96 opened this issue Jun 7, 2022 · 5 comments
Open

Memory leaks found while running tests. #1512

hovhannes-96 opened this issue Jun 7, 2022 · 5 comments

Comments

@hovhannes-96
Copy link

hovhannes-96 commented Jun 7, 2022

Memory leaks were found by LeakSanitizer.

  1. Build Klee with -fsanitize=leak flag.
  2. Run Klee's tests.
  3. The following tests fail and the LeakSanitizer reports memory leaks.
    test/Replay/klee-replay/KTestGen.c
    test/Replay/klee-replay/KTestRandGen.c
    test/regression/2020-02-24-count-paths-nodump.c
@hovhannes-96 hovhannes-96 changed the title Memory leak reported by LeakSanitizer. Memory leaks found while running tests. Jun 7, 2022
@hovhannes-96 hovhannes-96 reopened this Jun 7, 2022
@ccadar ccadar mentioned this issue Jun 13, 2022
8 tasks
@ccadar
Copy link
Contributor

ccadar commented Sep 24, 2022

@MartinNowack @251 I'm a bit confused as to why these tests pass on the ASan CI runs.
They fail on my machine.

@ccadar
Copy link
Contributor

ccadar commented Sep 24, 2022

To clarify: I haven't tried the CI scripts, but rather I see these errors being reported when I compile with -fsanitize=address or -fsanitize=leak.

@251
Copy link
Contributor

251 commented Sep 24, 2022

I'm a bit confused as to why these tests pass on the ASan CI runs.

Haven't checked the first ones but the last one disables asan explicitly for well-known reasons.

@MartinNowack
Copy link
Contributor

@ccadar Everything works as expected ;)

The reason why this still passes is that we explicitly ignore this issue: https://github.com/klee/klee/blob/master/utils/sanitizers/lsan.txt

@ccadar
Copy link
Contributor

ccadar commented Sep 26, 2022

OK, I haven't realised we have those ignore files...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants