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

[BUG] CUDTUnited::m_bGCStatus Requires a Lock #2976

Closed
maxsharabayko opened this issue Jul 9, 2024 · 0 comments · Fixed by #2990
Closed

[BUG] CUDTUnited::m_bGCStatus Requires a Lock #2976

maxsharabayko opened this issue Jul 9, 2024 · 0 comments · Fixed by #2990
Labels
[core] Area: Changes in SRT library core Type: Bug Indicates an unexpected problem or unintended behavior
Milestone
v1.5.4

Comments

@maxsharabayko
Copy link
Collaborator

maxsharabayko commented Jul 9, 2024 •
edited
Loading

Protect CUDTUnited::m_bGCStatus with m_InitLock from simultaneous access.

  • CUDTUnited::~CUDTUnited()
  • CUDT::socket()
  • CUDT::createGroup(SRT_GROUP_TYPE)

SRT v1.5.3.

Epic #1610 – Core Synchronization Issues.
@maxsharabayko maxsharabayko added Type: Bug Indicates an unexpected problem or unintended behavior [core] Area: Changes in SRT library core labels Jul 9, 2024
@maxsharabayko maxsharabayko added this to the v1.5.4 milestone Jul 9, 2024
@maxsharabayko maxsharabayko mentioned this issue Jul 9, 2024
Open
8 tasks
@ethouris ethouris mentioned this issue Jul 26, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
[core] Area: Changes in SRT library core Type: Bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Milestone
v1.5.4
Development

Successfully merging a pull request may close this issue.

[core] Put the use of m_bGCStatus under m_InitLock ethouris/srt
1 participant
@maxsharabayko