Skip to content

Commit

Permalink
Fix order of operations bug causing an assert.
Browse files Browse the repository at this point in the history
When we change our role (e.g. for symmetric connect) we may need to tear
down and re-init ICE.  The "locking" of config values was triggering an
assert when this happened.
  • Loading branch information
zpostfacto committed Aug 30, 2024
1 parent 04f7977 commit 1b57cce
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -255,10 +255,11 @@ void CSteamNetworkConnectionP2P::CheckInitICE()
#else
ICE_Implementation = 1;
#endif

m_connectionConfig.P2P_Transport_ICE_Implementation.Set( ICE_Implementation );
}

// Lock it in
m_connectionConfig.P2P_Transport_ICE_Implementation.Set( ICE_Implementation );
m_connectionConfig.P2P_Transport_ICE_Implementation.Lock();

// "Native" ICE client?
Expand Down

0 comments on commit 1b57cce

Please sign in to comment.