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.

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

Possible memory leak on client's free() #16

Open
vb-mich opened this issue Mar 18, 2020 · 3 comments
Open

Possible memory leak on client's free() #16

vb-mich opened this issue Mar 18, 2020 · 3 comments

Comments

@vb-mich
Copy link

vb-mich commented Mar 18, 2020

free(c) in wsclient.c, line 66, do not free any allocated memory on pointers inside wsclient struct (e.g URI, current_frame, etc..).
Am I correct?

@PixMod
Copy link

PixMod commented Mar 19, 2020

You are right. Though the URI is freed in line-28, but frames are leaked really.
Due to this project is not in maintains further, I have create https://github.com/morrowind/libwsclient/ to continue the project life. For your ref.

@mglonnro
Copy link

I tried to add @Morrowind's memory leak changes into the original code here: mglonnro@2352a62

@savethebeesandseeds
Copy link

@mglonnro Thank you for the help, ¿sure those changes are stable?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants