[Home]

Summary:ASTERISK-05462: janitor jump priority patch
Reporter:BJ Weschke (bweschke)Labels:
Date Opened:2005-11-04 16:47:39.000-0600Date Closed:2008-01-15 15:54:00.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_controlplayback
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) app_controlplayback_janitor.patch
Description: Channel status variable and j option added.
Comments:By: Kevin P. Fleming (kpfleming) 2005-11-04 17:15:54.000-0600

This doesn't look quite right... in the case where res < 0, there are paths where the result variable gets set but then res is not set back to zero... so the dialplan won't continue anyway.

According to this app's documentation, res should only be non-zero in the case where the channel was hung up during the playback; otherwise it should be zero, the result variable set, and optionally the priority changed to n+101.
By: BJ Weschke (bweschke) 2005-11-04 17:29:13.000-0600

you're right. fixed now.
By: Mark Spencer (markster) 2005-11-06 00:11:57.000-0600

Fixed in CVS head, thanks!
By: Digium Subversion (svnbot) 2008-01-15 15:54:00.000-0600

Repository: asterisk
Revision: 6971

U   trunk/apps/app_controlplayback.c

------------------------------------------------------------------------
r6971 | markster | 2008-01-15 15:54:00 -0600 (Tue, 15 Jan 2008) | 2 lines

Janitor project, app_controlplayback (bug ASTERISK-5462)

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

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