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

内存问题 #28

Closed
xstrings opened this issue Jul 13, 2014 · 4 comments
Closed

内存问题 #28

xstrings opened this issue Jul 13, 2014 · 4 comments
Assignees
@r12f
Labels
bug

Comments

@xstrings
Copy link

非常感谢libpe，为PE解析带来了极大的方便， 我这log发现DataLoaderDiskFile没有析构，带来了内存泄露，希望给予答复
@r12f
Copy link
Owner

r12f commented Jul 13, 2014

非常感谢你能告诉我这个消息，能有人真正在使用这个工程，我相当开心。我刚看到这个消息，让我检查一下代码，会尽快给你答复。另外请问你使用的是什么分支呢？default还是dev？

@r12f r12f added the bug label Jul 13, 2014
@r12f r12f self-assigned this Jul 13, 2014
@r12f
Copy link
Owner

r12f commented Jul 13, 2014

我找到问题的原因了，并且做了一个修正提交到了dev分支上，现在DataLoaderDiskFile已经能被正确析构了。有什么问题请随时联系我，多谢多谢。:D
2af0286

@xstrings
Copy link
Author

真没想到回复这么及时，现在已经能够正常使用了，修正很巧妙，再次感谢

@r12f
Copy link
Owner

r12f commented Jul 14, 2014

不谢不谢，应该的。也谢谢你告诉我这个问题。以后要是还遇到了任何问题，欢迎随时通知我～

@r12f r12f closed this as completed Jul 14, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@r12f r12f

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@r12f @xstrings