[Home]

Summary:ASTERISK-15388: mixmonitor CLI command is broken
Reporter:Jose (jlaguilar)Labels:
Date Opened:2010-01-02 21:30:12.000-0600Date Closed:2010-01-05 10:18:08.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_mixmonitor
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:In asterisk 1.6.2.0 the mixmonitor CLI command is being registered incorrectly.

app_mixmonitor.c (Line 593):
e->command = "mixmonitor {start|stop} {<chan_name>} [args]";

It is expecting the 3rd argument to be exactly "<chan_name>". If it isn't, asterisk will report that the command does not exist. And if you set it to be "<chan_name>" it will find the command but it will obviuosly return that the channel was not found.

Resgitering the command as:
e->command = "mixmonitor {start|stop}";

Fixes the problem.


Comments:By: Leif Madsen (lmadsen) 2010-01-04 09:51:04.000-0600

I've assigned this to mvanbaak for now, but if he can't fix it in the next few days, please let me know so I can reassign. Thanks!
By: Michiel van Baak (mvanbaak) 2010-01-05 09:27:46.000-0600

This is indeed the correct fix.
Thanks for reporting it, fixing it now.
By: Digium Subversion (svnbot) 2010-01-05 10:08:13.000-0600

Repository: asterisk
Revision: 237656

U   trunk/apps/app_mixmonitor.c

------------------------------------------------------------------------
r237656 | mvanbaak | 2010-01-05 10:08:12 -0600 (Tue, 05 Jan 2010) | 6 lines

Make CLI command 'mixmonitor start|stop <channel> work again.

(closes issue ASTERISK-15388)
Reported by: jlaguilar
Fix as suggested by jlaguilar in the bugreport

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

http://svn.digium.com/view/asterisk?view=rev&revision=237656
By: Digium Subversion (svnbot) 2010-01-05 10:09:38.000-0600

Repository: asterisk
Revision: 237657

_U  branches/1.6.2/
U   branches/1.6.2/apps/app_mixmonitor.c

------------------------------------------------------------------------
r237657 | mvanbaak | 2010-01-05 10:09:38 -0600 (Tue, 05 Jan 2010) | 13 lines

Merged revisions 237656 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r237656 | mvanbaak | 2010-01-05 17:08:12 +0100 (Tue, 05 Jan 2010) | 6 lines
 
 Make CLI command 'mixmonitor start|stop <channel> work again.
 
 (closes issue ASTERISK-15388)
 Reported by: jlaguilar
 Fix as suggested by jlaguilar in the bugreport
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=237657
By: Digium Subversion (svnbot) 2010-01-05 10:10:15.000-0600

Repository: asterisk
Revision: 237658

_U  branches/1.6.0/

------------------------------------------------------------------------
r237658 | mvanbaak | 2010-01-05 10:10:15 -0600 (Tue, 05 Jan 2010) | 12 lines

Blocked revisions 237656 via svnmerge

........
 r237656 | mvanbaak | 2010-01-05 17:08:12 +0100 (Tue, 05 Jan 2010) | 6 lines
 
 Make CLI command 'mixmonitor start|stop <channel> work again.
 
 (closes issue ASTERISK-15388)
 Reported by: jlaguilar
 Fix as suggested by jlaguilar in the bugreport
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=237658
By: Digium Subversion (svnbot) 2010-01-05 10:18:08.000-0600

Repository: asterisk
Revision: 237663

_U  branches/1.6.1/

------------------------------------------------------------------------
r237663 | mvanbaak | 2010-01-05 10:18:07 -0600 (Tue, 05 Jan 2010) | 12 lines

Blocked revisions 237656 via svnmerge

........
 r237656 | mvanbaak | 2010-01-05 17:08:12 +0100 (Tue, 05 Jan 2010) | 6 lines
 
 Make CLI command 'mixmonitor start|stop <channel> work again.
 
 (closes issue ASTERISK-15388)
 Reported by: jlaguilar
 Fix as suggested by jlaguilar in the bugreport
........

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

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