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

components/fs/kv/tos_kv.c中内存泄漏问题 #258

Closed
tluio opened this issue Dec 17, 2020 · 3 comments
Closed

components/fs/kv/tos_kv.c中内存泄漏问题 #258

tluio opened this issue Dec 17, 2020 · 3 comments

Comments

@tluio
Copy link
Contributor

tluio commented Dec 17, 2020

在tos_kv.c中发现了一些疑似内存泄漏的问题，不知道是否可以触发，建议检查一下
1、如果第344行和340行的if语句都为假，第329行为prev_key申请的内存会泄露
2、如果621行的item_out为NULL，并且从624行返回的话，第571行为item申请的内存以及第261行为kv_buf申请的内存会泄露
@Supowang1989
Copy link
Collaborator

感谢指出，我们近期修复

@tluio
Copy link
Contributor Author

tluio commented Jan 11, 2021

应该的，不客气！228也报告了一些疑似的内存泄漏问题，因为error handling code没处理好，您也可以一起看一下。

@daishengdong
Copy link
Collaborator

眼很尖！我马上提交修复

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@daishengdong @tluio @Supowang1989