Skip to content
This repository has been archived by the owner on May 16, 2024. It is now read-only.

fix potential mem leak #33

Merged
merged 2 commits into from
Oct 23, 2023
Merged

fix potential mem leak #33

merged 2 commits into from
Oct 23, 2023

Conversation

johnlanni
Copy link
Contributor

fix #31

@johnlanni
Copy link
Contributor Author

@anuraaga We should use GC_malloc_ignore_off_page when allocate large object without layout.

anuraaga
anuraaga reviewed Oct 23, 2023
View reviewed changes
Copy link
Contributor

@anuraaga anuraaga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

init.go Outdated Show resolved Hide resolved
@anuraaga anuraaga merged commit ecd0c03 into wasilibs:main Oct 23, 2023
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Reviewers

@anuraaga anuraaga anuraaga approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Memory leak exists, should use GC_malloc_ignore_off_page when alloc large object.
2 participants
@johnlanni @anuraaga