conncheck: socket reliability should not change the conncheck behaviour
authorFabrice Bellet <fabrice@bellet.info>
Sun, 3 May 2020 15:26:48 +0000 (17:26 +0200)
committerOlivier CrĂȘte <olivier.crete@ocrete.ca>
Thu, 7 May 2020 17:46:05 +0000 (17:46 +0000)
An inbound STUN request on a pair having another STUN request already
inflight already should generate to new triggered check, no matter the
type of the underlying socket.

agent/conncheck.c

index fc4a2b8..21a739a 100644 (file)
@@ -3231,7 +3231,7 @@ static gboolean priv_schedule_triggered_check (NiceAgent *agent, NiceStream *str
              * so there's no reason to recheck this pair, since it can in
              * no way replace the nominated one.
              */
-            if (!nice_socket_is_reliable (p->sockptr) && p->retransmit) {
+            if (p->retransmit) {
               nice_debug ("Agent %p : pair %p added for a triggered check.",
                   agent, p);
               priv_add_pair_to_triggered_check_queue (agent, p);