<p> Attention is currently required from: N A. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/19712">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">File channels/chan_iax2.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19712/comment/87aaf030_eed3ecdb">Patch Set #6, Line 4171:</a> <code style="font-family:monospace,monospace">                     ast_log(LOG_WARNING, "No voice format and no peer format available on %s, skipping frame\n", ast_channel_name(chan));</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">05:54:42  chan_iax2.c: In function '__get_from_jb':<br>05:54:42  chan_iax2.c:4171:114: error: 'chan' undeclared (first use in this function)<br>05:54:42      ast_log(LOG_WARNING, "No voice format and no peer format available on %s, skipping frame\n", ast_channel_name(chan));<br>05:54:42                                                                                                                    ^<br>05:54:42  chan_iax2.c:4171:114: note: each undeclared identifier is reported only once for each function it appears in<br>05:54:42  make[1]: *** [chan_iax2.o] Error 1</p><p style="white-space: pre-wrap; word-wrap: break-word;">__get_from_jb doesn't receive the chan from the caller.  pvt->owner I believe should be the one you're looking for.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19712/comment/48f0d22a_aff75e5d">Patch Set #6, Line 4215:</a> <code style="font-family:monospace,monospace">cleanup:</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">OK, so here I have a question:</p><p style="white-space: pre-wrap; word-wrap: break-word;">Won't this cause a blocking until voiceformat does pass, ie, it won't actually jb_get the next frames, as in ever, or at least until voiceformat does get set, and then burst them?  Which is why even without early media one won't get RINGING on the caller side.  In other words, if the peerformat is also not set, the current bug will persist.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I do think there are bigger (configuration problems if this is the case, so not too worried, I think this shall be "good enough").  But in reality we're not actually dropping the frame, we're backlogging it.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/19712">change 19712</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/19712"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ie4fd1e8e088a145ad89e0427c2100a530e964fe9 </div>
<div style="display:none"> Gerrit-Change-Number: 19712 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: N A <asterisk@phreaknet.org> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-CC: Jaco Kroon <jaco@uls.co.za> </div>
<div style="display:none"> Gerrit-Attention: N A <asterisk@phreaknet.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 31 Jan 2023 12:13:02 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>