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.

Sign up for GitHub

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

Jump to bottom

Prevent memory leak #73

Closed
hangpark opened this issue Apr 12, 2017 · 1 comment
Closed

Prevent memory leak #73

hangpark opened this issue Apr 12, 2017 · 1 comment
Assignees
@hangpark
Labels
bug
Milestone
Project #2

Comments

@hangpark
Copy link
Owner

hangpark commented Apr 12, 2017 •
edited
Loading

It seems memory leaks are occurring. Every test is passed except multi-oom, whose role is detecting memory leaks (in both bochs and qemu.)

Result of multi-oom:

run: after run 1/10, expected depth 61, actual depth 44: FAILED
@hangpark hangpark added the bug label Apr 12, 2017
@hangpark hangpark added this to the Project #2 milestone Apr 12, 2017
@hangpark hangpark self-assigned this Apr 12, 2017
hangpark added a commit that referenced this issue Apr 13, 2017
@hangpark
[#73] Avoid memory leak
4dfa242 
Free resources that had not been controlled.
hangpark added a commit that referenced this issue Apr 13, 2017
@hangpark
Merge pull request #74 from hangpark/iss/73
9667427 
[#73] Avoid memory leak
@hangpark hangpark mentioned this issue Apr 13, 2017
Merged
@hangpark
Copy link
Owner Author

Done by PR #74.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hangpark hangpark

Labels
bug
Projects
None yet
Milestone
Project #2
Development

No branches or pull requests
1 participant
@hangpark