Commit ef95b2e1 authored by Fabrice Bellet's avatar Fabrice Bellet Committed by Olivier Crête

conncheck: improve places where the conncheck list is dumped

parent cbf0162d
...@@ -475,12 +475,11 @@ void ...@@ -475,12 +475,11 @@ void
conn_check_unfreeze_related (NiceAgent *agent, CandidateCheckPair *pair) conn_check_unfreeze_related (NiceAgent *agent, CandidateCheckPair *pair)
{ {
GSList *i, *j; GSList *i, *j;
gboolean result = FALSE;
g_assert (pair); g_assert (pair);
g_assert (pair->state == NICE_CHECK_SUCCEEDED); g_assert (pair->state == NICE_CHECK_SUCCEEDED);
priv_print_conn_check_lists (agent, G_STRFUNC, NULL);
for (i = agent->streams; i ; i = i->next) { for (i = agent->streams; i ; i = i->next) {
NiceStream *s = i->data; NiceStream *s = i->data;
for (j = s->conncheck_list; j ; j = j->next) { for (j = s->conncheck_list; j ; j = j->next) {
...@@ -495,9 +494,15 @@ conn_check_unfreeze_related (NiceAgent *agent, CandidateCheckPair *pair) ...@@ -495,9 +494,15 @@ conn_check_unfreeze_related (NiceAgent *agent, CandidateCheckPair *pair)
nice_debug ("Agent %p : Unfreezing check %p " nice_debug ("Agent %p : Unfreezing check %p "
"(after successful check %p).", agent, p, pair); "(after successful check %p).", agent, p, pair);
SET_PAIR_STATE (agent, p, NICE_CHECK_WAITING); SET_PAIR_STATE (agent, p, NICE_CHECK_WAITING);
result = TRUE;
} }
} }
} }
/* We dump the conncheck list when something interesting happened, ie
* when we unfroze some pairs.
*/
if (result)
priv_print_conn_check_lists (agent, G_STRFUNC, NULL);
} }
/* /*
...@@ -514,12 +519,11 @@ static void ...@@ -514,12 +519,11 @@ static void
priv_conn_check_unfreeze_maybe (NiceAgent *agent, CandidateCheckPair *pair) priv_conn_check_unfreeze_maybe (NiceAgent *agent, CandidateCheckPair *pair)
{ {
GSList *i, *j; GSList *i, *j;
gboolean result = FALSE;
g_assert (pair); g_assert (pair);
g_assert (pair->state == NICE_CHECK_FROZEN); g_assert (pair->state == NICE_CHECK_FROZEN);
priv_print_conn_check_lists (agent, G_STRFUNC, NULL);
for (i = agent->streams; i ; i = i->next) { for (i = agent->streams; i ; i = i->next) {
NiceStream *s = i->data; NiceStream *s = i->data;
for (j = s->conncheck_list; j ; j = j->next) { for (j = s->conncheck_list; j ; j = j->next) {
...@@ -531,9 +535,15 @@ priv_conn_check_unfreeze_maybe (NiceAgent *agent, CandidateCheckPair *pair) ...@@ -531,9 +535,15 @@ priv_conn_check_unfreeze_maybe (NiceAgent *agent, CandidateCheckPair *pair)
nice_debug ("Agent %p : Unfreezing check %p " nice_debug ("Agent %p : Unfreezing check %p "
"(after successful check %p).", agent, pair, p); "(after successful check %p).", agent, pair, p);
SET_PAIR_STATE (agent, pair, NICE_CHECK_WAITING); SET_PAIR_STATE (agent, pair, NICE_CHECK_WAITING);
result = TRUE;
} }
} }
} }
/* We dump the conncheck list when something interesting happened, ie
* when we unfroze some pairs.
*/
if (result)
priv_print_conn_check_lists (agent, G_STRFUNC, NULL);
} }
guint guint
...@@ -773,9 +783,9 @@ timer_return_timeout: ...@@ -773,9 +783,9 @@ timer_return_timeout:
} }
if (pair) { if (pair) {
priv_print_conn_check_lists (agent, G_STRFUNC,
", got a pair in Waiting state");
priv_conn_check_initiate (agent, pair); priv_conn_check_initiate (agent, pair);
priv_print_conn_check_lists (agent, G_STRFUNC,
", initiated an ordinary connection check");
return TRUE; return TRUE;
} }
...@@ -1152,9 +1162,10 @@ static gboolean priv_conn_check_tick_agent_locked (NiceAgent *agent, ...@@ -1152,9 +1162,10 @@ static gboolean priv_conn_check_tick_agent_locked (NiceAgent *agent,
pair = priv_get_pair_from_triggered_check_queue (agent); pair = priv_get_pair_from_triggered_check_queue (agent);
if (pair) { if (pair) {
int result = conn_check_send (agent, pair);
priv_print_conn_check_lists (agent, G_STRFUNC, priv_print_conn_check_lists (agent, G_STRFUNC,
", got a pair from triggered check list"); ", initiated a connection check from triggered check list");
if (conn_check_send (agent, pair)) { if (result) {
SET_PAIR_STATE (agent, pair, NICE_CHECK_FAILED); SET_PAIR_STATE (agent, pair, NICE_CHECK_FAILED);
return FALSE; return FALSE;
} }
......
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