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

memleak when set user in config file. #1564

Closed
back1127 opened this issue Jan 20, 2020 · 1 comment
Closed

memleak when set user in config file. #1564

back1127 opened this issue Jan 20, 2020 · 1 comment
Milestone
1.6.9

Comments

@back1127
Copy link

if set user in config file, it will strdup user to config->user. but don't free when call config__cleanup.
@ralight
Copy link
Contributor

ralight commented Jan 28, 2020

Thanks, keep on looking for these!

@ralight ralight added this to the 1.6.9 milestone Jan 28, 2020
@ralight ralight closed this as completed Jan 28, 2020
ralight added a commit that referenced this issue Jan 28, 2020
@ralight
Fix config->user not being freed on exit.
e3463b0 
Closes #1564. Thanks to back1127.
ralight added a commit that referenced this issue Jan 28, 2020
@ralight
Fix config->user not being freed on exit.
9ae3878 
Closes #1564. Thanks to back1127.
@lock lock bot locked as resolved and limited conversation to collaborators May 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.6.9
Development

No branches or pull requests
2 participants
@ralight @back1127