[Asterisk-code-review] app_dial: Prevent call from hanging (asterisk[master])

Joshua Colp asteriskteam at digium.com
Thu Jun 10 08:58:53 CDT 2021


Attention is currently required from: N A.
Joshua Colp has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/15985 )

Change subject: app_dial: Prevent call from hanging
......................................................................


Patch Set 1:

(1 comment)

File apps/app_dial.c:

https://gerrit.asterisk.org/c/asterisk/+/15985/comment/b592f1a0_61cba9c8 
PS1, Line 1801: 						/* If we're trying to send DTMF and the call has hung up
> That's exactly the thing, everywhere I looked, as far as Asterisk was concerned, it thought the chan […]
A channel is up until it is hung up, which occurs as a result of a hangup being queued on the channel or flags being set (which then trigger the channel to be read to see that it is hung up) which then has the channel hung up and then ultimately destroyed. That should be investigated, specifically seeing if the hangup has been queued, and then looking at the handling in app_dial.



-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/15985
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I6f2ee3f77b892015bc05513d868f071f279a3c80
Gerrit-Change-Number: 15985
Gerrit-PatchSet: 1
Gerrit-Owner: N A <mail at interlinked.x10host.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Attention: N A <mail at interlinked.x10host.com>
Gerrit-Comment-Date: Thu, 10 Jun 2021 13:58:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: N A <mail at interlinked.x10host.com>
Comment-In-Reply-To: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210610/bf284d61/attachment.html>


More information about the asterisk-code-review mailing list