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

Bug27795 27782 #364

Merged
5 commits merged into from Sep 20, 2018
Merged

Bug27795 27782 #364

5 commits merged into from Sep 20, 2018

Conversation

nmathewson
Copy link
Contributor

No description provided.

@nmathewson
Add a tor_release_socket_ownership() function.
1c1e842
@nmathewson
Fix socket accounting with ORConn sockets.
f25323f 
When we close a socket via tor_tls_free(), we previously had no way
for our socket accounting logic to learn about it.  This meant that
the socket accounting code would think we had run out of sockets,
and freak out.

Fixes bug 27795; bugfix on 0.3.5.1-alpha.
@nmathewson
Release ownership of the dummy socket that tortls_nss.c will close
62b709b 
Related to #27795 -- since NSS will close the socket, we must not
count it as owned by Tor.
@nmathewson
Use the correct macro to detect an invalid socket in tortls_nss.c
d6c564e 
Fixes bug 27782; bugfix on 0.3.5.1-alpha
@coveralls
Copy link

coveralls commented Sep 20, 2018 •
edited

Coverage Status

Coverage decreased (-0.003%) to 61.89% when pulling 4e20281 on nmathewson:bug27795_27782 into bd6007d on torproject:master.

@ghost ghost merged commit 4e20281 into torproject:master Sep 20, 2018
This pull request was closed.
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
2 participants
@nmathewson @coveralls