search

nanomsg / nng

nanomsg-next-generation -- light-weight brokerless messaging
https://nng.nanomsg.org
MIT License
3.78k stars 485 forks source link

tls: reap the connection mutex #1810

Closed gdamore closed 5 months ago

gdamore commented 5 months ago

On some systems mutexes acquire memory, and we need to make sure finalize the mutex.

This was reported by Jaylin Yu.

fixes #

Note that the above format should be used in your git commit comments. You agree that by submitting a PR, you have read and agreed to our contributing guidelines.
codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.34%. Comparing base (b1aaa60) to head (b16157c).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1810 +/- ## ======================================= Coverage 79.34% 79.34% ======================================= Files 95 95 Lines 21328 21329 +1 ======================================= + Hits 16922 16924 +2 + Misses 4406 4405 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

gdamore commented 5 months ago

Closing in favor of #1811