Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Game crashes with more than 2 connections when using PixelStreaming with SFU #62

Closed
ashishdasari148 opened this issue Jan 16, 2023 · 2 comments · Fixed by #129
Closed

Game crashes with more than 2 connections when using PixelStreaming with SFU #62

ashishdasari148 opened this issue Jan 16, 2023 · 2 comments · Fixed by #129

Comments

@ashishdasari148
Copy link

ashishdasari148 commented Jan 16, 2023 •
edited
Loading

With reference implementations of the Signalling server and the SFU, when trying to connect from the 3rd tab, the game crashes. This issue is not reproducible if ?preferSFU=false.

Signal 11 caught.
Malloc Size=262146 LargeMemoryPoolOffset=262162
CommonUnixCrashHandler: Signal=11
[2023.01.16-12.13.11:591][244]LogCore: === Critical error: ===
Unhandled Exception: SIGSEGV: invalid attempt to write memory at address 0x0000000000000003

[2023.01.16-12.13.11:591][244]LogCore: Assertion failed: RecvResult.ok() [File:.\../Plugins/Media/PixelStreaming/Source/PixelStreaming/Private/PixelStreamingDataChannel.cpp] [Line: 52]
Failed to create Data Channel. Msg=Data channel creation failed
@lukehb
Copy link
Contributor

lukehb commented Feb 27, 2023

We are working on this one. CC @StomyPX if you end up with a fix in hand, make sure we share relevant details on this ticket too.

@StomyPX StomyPX mentioned this issue Feb 28, 2023
Merged
lukehb pushed a commit that referenced this issue Feb 28, 2023
@StomyPX
Use mediasoup's internal stream ids for SCTP to avoid clashes (#129)
c82e13b 
This should allow more than 2 peers to connect to the SFU without having
Unreal Engine crash from stream channels being double-assigned.

This addresses: #62
@lukehb
Copy link
Contributor

lukehb commented Feb 28, 2023

This is fixed in c82e13b

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use mediasoup's internal stream ids for SCTP to avoid clashes StomyPX/PixelStreamingInfrastructure
2 participants
@lukehb @ashishdasari148