-
Notifications
You must be signed in to change notification settings - Fork 41
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
Implementing Close DataChannel #20
Comments
It should be, but that would be well worth a double-check when implementing. Thanks for the reminder, I'll try to take a look at implementing this, or you're more than welcome to submit a PR :) |
Ah okay, I'm looking at it too. I'm trying to ensure that the testclient program could terminate safely so I could then check for any memory leaks with valgrind. |
Perfect - one challenge I've had with Valgrind is that it gets super upset with OpenSSL. You can create ignore lists for a lot of the OpenSSL warnings that should help sort the signal from the noise. |
Yep! I've seen that. Crypto functions in OpenSSL use uninitialized memory on purpose for the entropy. I've used suppression files for this before. Anyway, here's my attempt at solving this. I'm trying to reset the stream using |
I think I got this working. Will send a PR soon once I do a bit of clean up :) |
PR: #24 |
I've noticed that Close() is a TODO at the moment.
To implement this, is it sufficient to use
usrsctp_deregister_address()
as it's done here?The text was updated successfully, but these errors were encountered: