[Home]

Summary:ASTERISK-00754: [patch] Using Announce feature of Dial app may have unexpected side affects!
Reporter:Steve Murphy (murf)Labels:
Date Opened:2004-01-07 13:40:15.000-0600Date Closed:2008-01-15 14:41:42.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) diffu.app_dial.announcefix
Description:The file name used in the A(file) option of the dial
command is not XXX-d out like the P() args are. Therefore
they affect the option scans. The unlucky user who
specifies A(Troublemsg_tdc) or somesuch, may not realize
they have just specified the t,T,r,m,d,c, and g options to
Dial. I've included some diffs that fix this problem.

****** ADDITIONAL INFORMATION ******

Also, using the P (privacy) or C options excludes the A() option. I do not see any reason for this. As a matter of fact, I see that using the A() to let the called person know they are connected to the calling party can be mighty handy! The attached diff -u patch also address this problem. As a matter of fact, the Announce args MUST be X'd out, or they will affect the options. You can't hide them by using another option!
Comments:By: Mark Spencer (markster) 2004-01-15 14:28:55.000-0600

Fixed in CVS with modified version of patch
By: Digium Subversion (svnbot) 2008-01-15 14:41:42.000-0600

Repository: asterisk
Revision: 2039

U   trunk/apps/app_dial.c

------------------------------------------------------------------------
r2039 | markster | 2008-01-15 14:41:41 -0600 (Tue, 15 Jan 2008) | 2 lines

X out announce stuff (bug ASTERISK-754)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=2039