<html><head></head><body><div>Hi,</div><div><br></div><div>Merged those commits as it was blocked on me, I finally took time to review them and merge them straight to the git master. So if you pull now you should have them. Please test carefully as there are a lot of changes!</div><div><br></div><div>Olivier</div><div><br></div><div>On Fri, 2018-03-23 at 18:40 +0200, Igor Khomenko wrote:</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div dir="ltr"><div><div><div>Thanks Fabrice for getting back to me<br><br></div>is there a custom Git branch with the patch I can build from or should I port all those changes manually?<br><br></div>At this moment I use latest master - commits on Nov 28, 2017
<br><br></div>thanks<br><div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 23 March 2018 at 18:32, Fabrice Bellet <span dir="ltr"><<a href="mailto:fabrice@bellet.info" target="_blank">fabrice@bellet.info</a>></span> wrote:<br><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex">Hi Igor,<br>
<span class=""><br>
On 03/22/18 at 10:12am, Igor Khomenko wrote:<br>
> Hi Olivier, Lorenzo<br>
><br>
> I've got all needed logs upon crash, including libnice logs<br>
><br>
> Stacktrace again (the same assert):<br>
> <a href="https://www.dropbox.com/s/jlqtzu0neqlqc8o/janus_libnice_assert_stacktrace.crash?dl=0" rel="noreferrer" target="_blank">https://www.dropbox.com/s/<wbr>jlqtzu0neqlqc8o/janus_libnice_<wbr>assert_stacktrace.crash?dl=0</a><br>
><br>
> logs:<br>
> <a href="https://www.dropbox.com/s/hq1nkeh6w25jsk3/janus.log?dl=0" rel="noreferrer" target="_blank">https://www.dropbox.com/s/<wbr>hq1nkeh6w25jsk3/janus.log?dl=0</a><br>
><br>
> look at the end of janus.log<br>
> Line N19365 - "(process:17784): libnice-DEBUG: Agent 0x7f6aa000dcf0 :<br>
> stream 1 component 1 STATE-CHANGE gathering -> connected."<br>
> this is when crash happened, then autorestart of Janus occurred after some<br>
> time.<br>
><br>
> So all the lines before N19365 are what we are looking for from my<br>
> understanding,<br>
> e.g. line N19330 is where assertion failed<br>
><br>
> Please let me know if you need anything else here to identify the root<br>
> cause and find a proper solution<br>
<br>
</span>According to your logs, I think the cause of the problem is that no pair<br>
is formed by the gathering stage: we have two local IPv4 candidates, and<br>
we receive a remote IPv6 candidate. A new pair is formed later when we<br>
start receiving incoming STUN requests. The state of the agent should<br>
change from GATHERING to CONNECTING there, line 19285 in your janus.log<br>
file:<br>
<br>
(process:17784): libnice-DEBUG: Agent 0x7f6aa000dcf0 : Adding a triggered check to conn.check list (local=0x7f6aa00239f0).<br>
(process:17784): libnice-DEBUG: Agent 0x7f6aa000dcf0 : creating new pair 0x7f6aa001a190 state 1<br>
<br>
Does this patch help in your case : <a href="https://phabricator.freedesktop.org/D1889" rel="noreferrer" target="_blank">https://phabricator.<wbr>freedesktop.org/D1889</a><br>
And more precisely, the chunk around line 2967-2968 in conncheck.c that replaces<br>
priv_add_new_check_pair() by priv_conn_check_add_for_<wbr>candidate_pair_matched()<br>
at the end of function priv_schedule_triggered_check(<wbr>) ?<br>
<br>
Best wishes,<br>
<span class="HOEnZb"><font color="#888888">--<br>
fabrice<br>
</font></span></blockquote></div><br><br clear="all"><br></div></blockquote><div><span><pre><pre>-- <br></pre>Olivier Crête
olivier.crete@collabora.com
</pre></span></div></body></html>