<p> Attention is currently required from: N A. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/15985">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">File apps/app_dial.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/+/15985/comment/26be3f72_a663c76e">Patch Set #2, Line 1802:</a> <code style="font-family:monospace,monospace">                                           if (ast_check_hangup_locked(o->chan)) {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">The existing logic fails because it's possible for a channel (that exists) to have a soft hangup que […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">It should not be necessary for app_dial to call ast_check_hangup_locked, ast_read should be called and it should return NULL. The act of setting a softhangup flag queues a frame on the channel to cause what is handling the channel to wake up and call ast_read, which then returns NULL. The ast_read code in fact has code that if ast_check_hangup returns true then it skips its logic, and causes NULL to be returned every time you call ast_read on the channel. The ast_read function has to be called on the channel though for that to happen - so evidently in this case, it's not. The question is: why? Is something else handling the channel at the time and the hangup is dropped as a result of that? Is another channel getting priority for reading so the other channel is never read due to some situation?</p><p style="white-space: pre-wrap; word-wrap: break-word;">As for why ast_check_hangup and the other associated things exist, it's dependent on how the channel is being handled. There are valid cases where it is useful. Does it fix the issue you see? Sure. Should it be necessary? I'm not convinced.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/15985">change 15985</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/+/15985"/><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: I6f2ee3f77b892015bc05513d868f071f279a3c80 </div>
<div style="display:none"> Gerrit-Change-Number: 15985 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: N A <mail@interlinked.x10host.com> </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-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 06 Aug 2021 23:35:55 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Comment-In-Reply-To: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>