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

Memory leak on easy and multi #15

Open
ypsik opened this issue Dec 9, 2019 · 1 comment
Open

Memory leak on easy and multi #15

ypsik opened this issue Dec 9, 2019 · 1 comment

Comments

@ypsik
Copy link

ypsik commented Dec 9, 2019

Hello,

SafeEasyHandler is defined as
private SafeEasyHandle() : base(IntPtr.Zero, false)

So handle isnt owed by SafeHandle and ReleaseHandle will be not called.
Then CurlNative.Easy.Cleanup(handle); missed in all examples (for multi is the same ).
Or just change to
private SafeEasyHandle() : base(IntPtr.Zero, true)

@Werner-Prbk
Copy link

Is there any update on this?
I can confirm that there is a memory leak and would also suggest to change the implementation to

private SafeEasyHandle() : base(IntPtr.Zero, true)

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

2 participants