[Asterisk-code-review] app_dial.c: Only send DTMF on first progress event. (asterisk[16])

[Joshua Colp]

Joshua Colp has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/15593 )

Change subject: app_dial.c: Only send DTMF on first progress event.
......................................................................

app_dial.c: Only send DTMF on first progress event.

ASTERISK-29329 #close

Change-Id: Ic58e7a17f1ff3f785a5b21dced88682581149601
---
M apps/app_dial.c
1 file changed, 8 insertions(+), 6 deletions(-)

Approvals:
  Joshua Colp: Looks good to me, but someone else must approve; Approved for Submit
  George Joseph: Looks good to me, approved



diff --git a/apps/app_dial.c b/apps/app_dial.c
index 2744868..b3dceca 100644
--- a/apps/app_dial.c
+++ b/apps/app_dial.c
@@ -1543,12 +1543,14 @@
 						ast_channel_stage_snapshot_done(in);
 						ast_channel_unlock(in);
 						sent_progress = 1;
-					}
-					if (!ast_strlen_zero(dtmf_progress)) {
-						ast_verb(3,
-							"Sending DTMF '%s' to the called party as result of receiving a PROGRESS message.\n",
-							dtmf_progress);
-						ast_dtmf_stream(c, in, dtmf_progress, 250, 0);
+
+						if (!ast_strlen_zero(dtmf_progress)) {
+							ast_verb(3,
+								"Sending DTMF '%s' to the called party as result of "
+								"receiving a PROGRESS message.\n",
+								dtmf_progress);
+							ast_dtmf_stream(c, in, dtmf_progress, 250, 0);
+						}
 					}
 					ast_channel_publish_dial(in, c, NULL, "PROGRESS");
 					break;

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

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: Ic58e7a17f1ff3f785a5b21dced88682581149601
Gerrit-Change-Number: 15593
Gerrit-PatchSet: 2
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210310/94eabe36/attachment.html>