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

array thread safety wrapper to fix racing condition causing Fatal error #156

Merged
merged 1 commit into from
Feb 7, 2024

Conversation

kalafus
Copy link
Contributor

@kalafus kalafus commented Feb 5, 2024

thread safe array wrapper to fix racing condition

What

Fatal error: UnsafeMutablePointer.moveUpdate(from:) with negative count at self?.streamingSessions.removeAll(where: { $0 == object })

Why

array can be concurrently accessed with append and removeAll methods.

Affected Areas

networking

…ror: UnsafeMutablePointer.moveUpdate(from:) with negative count` at `self?.streamingSessions.removeAll(where: { $0 == object })`
Copy link

sonarcloud bot commented Feb 5, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@ingvarus-bc
Copy link
Contributor

Nice job! Thanks for your PR!

@ingvarus-bc ingvarus-bc merged commit 04ee92c into MacPaw:main Feb 7, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants