[asterisk-commits] mjordan: branch 1.8 r381916 - /branches/1.8/apps/app_parkandannounce.c

[SVN commits to the Asterisk project]

Author: mjordan
Date: Sun Feb 24 09:33:32 2013
New Revision: 381916

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=381916
Log:
Make ParkAndAnnounce return to priority + 1 when return context is not defined

The ParkAndAnnounce application documentation for the optional return_context
parameter states the following:

return_context
    The goto-style label to jump the call back into after timeout. Default
    'priority+1'.

Unfortunately, the application was sending the channel back into the dialplan
at 'priority', which is the ParkAndAnnounce application call. This causes an
infinite loop of the channel constantly being parked, announced, timed out,
parked, announced, timed out... while fun, especially for those callers you
wish to drive to the end of madness, this was not the intent of the
application.

(closes issue ASTERISK-20113)
Reported by: serginuez
patches:
  app_parkandannounce.diff uploaded by serginuez (License 6405)


Modified:
    branches/1.8/apps/app_parkandannounce.c

Modified: branches/1.8/apps/app_parkandannounce.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/apps/app_parkandannounce.c?view=diff&rev=381916&r1=381915&r2=381916
==============================================================================
--- branches/1.8/apps/app_parkandannounce.c (original)
+++ branches/1.8/apps/app_parkandannounce.c Sun Feb 24 09:33:32 2013
@@ -129,6 +129,8 @@
 	if (!ast_strlen_zero(args.return_context)) {
 		ast_clear_flag(chan, AST_FLAG_IN_AUTOLOOP);
 		ast_parseable_goto(chan, args.return_context);
+	} else {
+		chan->priority++;
 	}
 
 	ast_verb(3, "Return Context: (%s,%s,%d) ID: %s\n", chan->context, chan->exten,