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 leak in rt.minfo according to ASAN #4675

Closed
rikkimax opened this issue Jun 2, 2024 · 3 comments
Closed

Memory leak in rt.minfo according to ASAN #4675

rikkimax opened this issue Jun 2, 2024 · 3 comments

Comments

@rikkimax
Copy link
Contributor

rikkimax commented Jun 2, 2024

Ldc 1.37.0 on Linux.

Using address sanitizer enabled for a shared library, it appears that there may be a memory leak in rt.minfo.

While it may be an upstream bug, I'm reporting here first because its ASAN that is catching it.

==57995==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 80 byte(s) in 2 object(s) allocated from:
    #0 0x7f72159386b6 in realloc /home/runner/work/llvm-project/llvm-project/compiler-rt/lib/asan/asan_malloc_linux.cpp:85:3
    #1 0x561f327a4e13 in _D2rt5minfo11ModuleGroup9sortCtorsMFNbAyaZ6doSortMFNbmKAPyS6object10ModuleInfoZb (/home/rikki/projects/ProjectSidero/eventloop/examples/networking/example_networking+0xb0e13) (BuildId: 08ee12abadfb191c7201255e72848f7ccb6710e9)

SUMMARY: AddressSanitizer: 80 byte(s) leaked in 2 allocation(s).
@kinke
Copy link
Member

kinke commented Jun 2, 2024

While it may be an upstream bug, I'm reporting here first because its ASAN that is catching it.

Not sure I get the rationale wrt. ASan and then filing here, the 'issue' is almost certainly upstream, just like other little leaks in #4617.

@rikkimax
Copy link
Contributor Author

rikkimax commented Jun 2, 2024

That is my instinct too, but I thought it was better to report here first to make sure there is no additional context that I may not be aware of.

@rikkimax rikkimax closed this as completed Jun 2, 2024
@rikkimax
Copy link
Contributor Author

rikkimax commented Jun 2, 2024

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

2 participants