Commit 17488a20 authored by Philip Withnall's avatar Philip Withnall

conncheck: foundations are shared across streams

This patch fixes a bug where the foundation definition shouldn't take
into account the stream the pair belongs to. This is important, because
the ordinary checks algorithm will change pair state from Frozen to
Waiting, by selecting pairs from other streams sharing the same
foundation than already succeeded pairs.
Reviewed-by: default avatarOlivier Crête <olivier.crete@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D815
parent 3ce45c25
...@@ -393,7 +393,6 @@ static void priv_assign_foundation (NiceAgent *agent, NiceCandidate *candidate) ...@@ -393,7 +393,6 @@ static void priv_assign_foundation (NiceAgent *agent, NiceCandidate *candidate)
if (candidate->type == n->type && if (candidate->type == n->type &&
candidate->transport == n->transport && candidate->transport == n->transport &&
candidate->stream_id == n->stream_id &&
nice_address_equal_no_port (&candidate->base_addr, &n->base_addr) && nice_address_equal_no_port (&candidate->base_addr, &n->base_addr) &&
(candidate->type != NICE_CANDIDATE_TYPE_RELAYED || (candidate->type != NICE_CANDIDATE_TYPE_RELAYED ||
priv_compare_turn_servers (candidate->turn, n->turn)) && priv_compare_turn_servers (candidate->turn, n->turn)) &&
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment