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

Add tracing mmap and munmap functions #38

Open
Bojun-Seo opened this issue Nov 20, 2024 · 2 comments
Open

Add tracing mmap and munmap functions #38

Bojun-Seo opened this issue Nov 20, 2024 · 2 comments

Comments

@Bojun-Seo
Copy link
Collaborator

Current version of heaptrace doesn't trace mmap and munmap functions.
Add tracing these functions to detect memory leak caused by them.

@Bojun-Seo
Copy link
Collaborator Author

@honggyukim
You removed this feature since the implementation was not mature enough.
32163c3
Can you tell me what was not mature? if you remember?

@honggyukim
Copy link
Owner

As far as I remember, the mmap hooking was removed because the behaviors of mmap heavily depends on input flags so it's not simple enough to handle it.

We may have to handle only specific flags such as MAP_ANON, etc and ignore the rest.

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