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

Changed DTLS transport to use recursive mutex, instead of a simple one #3731

Merged
merged 1 commit into from
Oct 9, 2023

Conversation

sauwming
Copy link
Member

@sauwming sauwming commented Oct 9, 2023

As reported in the comment 6dc9b8c#r129061968
we should use recursive mutex for DTLS transport, instead of simple one.

Note that all our media transports are currently already equipped with a group lock, so this change should affect only those who implement their own custom media transport without utilising the group lock. To avoid deadlock, app is recommended to use group lock instead.

@sauwming sauwming self-assigned this Oct 9, 2023
@sauwming sauwming added this to the release-2.14 milestone Oct 9, 2023
@sauwming sauwming merged commit eaf0f45 into master Oct 9, 2023
30 of 34 checks passed
@sauwming sauwming deleted the dtls-mutex branch October 9, 2023 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants