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

Fix memory leak when Organization Search lookup #18

Closed
wants to merge 1 commit into from
Closed

Fix memory leak when Organization Search lookup #18

wants to merge 1 commit into from

Conversation

shojikai
Copy link

I found that Organization Serach has caused a memory leak.
I fixed it, please check.

alindeman added a commit that referenced this pull request Oct 20, 2014
@alindeman
Makes sure to free a pointer we own
bd1956d 
Fixes #18
@alindeman alindeman mentioned this pull request Oct 20, 2014
Open
@alindeman
Copy link
Contributor

Hi @shojikai: I think the free should be done closer to the place where the memory is assigned/allocated. What do you think of #19?

alindeman added a commit that referenced this pull request Oct 20, 2014
@alindeman
Makes sure to free a pointer we own
7827bfd 
Fixes #18
@shojikai
Copy link
Author

I think good.
Thank you.

alindeman added a commit that referenced this pull request Oct 22, 2014
@alindeman
Makes sure to free a pointer we own
38a79ad 
Fixes #18
@shojikai shojikai closed this by deleting the head repository Sep 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@shojikai @alindeman