Commit fc4d3aab authored by Olivier Crête's avatar Olivier Crête

agent: ignore gathering failures on auto-generated IPs

Candidate gathering is stopped when discovery_add_local_host_candidate()
returns HOST_CANDIDATE_CANT_CREATE_SOCKET. This may be too radical
a measure when other local addresses can still be able to generate
usable candidates.

The issue was observed by a user who had an IPv6 address with tentative
flag on one of the interfaces. That single failing address was causing
the whole gathering process to end with no candidates found.

Still, don't do this if nice_agent_add_local_address() has been called.
In that case, the user really cares about the addresses and if there's
any problem, the process should fail.

https://phabricator.freedesktop.org/D1016
parent 1fb6401d
...@@ -2786,6 +2786,10 @@ nice_agent_gather_candidates ( ...@@ -2786,6 +2786,10 @@ nice_agent_gather_candidates (
" s%d:%d. Invalid interface?", agent, ip, stream->id, " s%d:%d. Invalid interface?", agent, ip, stream->id,
component->id); component->id);
} }
if (agent->local_addresses == NULL) {
/* Ignore when an auto-generated address fails. */
continue;
}
ret = FALSE; ret = FALSE;
goto error; goto error;
} }
......
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