[Home]

Summary:ASTERISK-05980: [patch] certain meetme admin/user menu options cause audio problems
Reporter:gkloepfer (gkloepfer)Labels:
Date Opened:2006-01-04 12:52:55.000-0600Date Closed:2008-01-15 16:44:45.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_meetme
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) menus-bugfix
Description:When certain selections are made (specifically, 4 6 and 8 will do this) from the admin or user menus during a meetme conference, many errors of the form:

Dec 20 13:39:45 WARNING[29627]: codec_gsm.c:165 gsmtolin_framein: Huh?  A GSM frame that isn't a multiple of 33 or 65 bytes long from (null) (320)?

appear, or the audio in the conference is permanently muted for that user.

These problems appear to happen because the ast_waitstream() that follows the ast_streamfile() doesn't stop the file from streaming when a keypress is received.  This results in confusion in the channel when the mu-law data from the resumed conference is placed on the channel.

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

I asked for feedback on the mailing list and didn't receive any, so I am reporting this as a bug since it is repeatable and this fix appears correct.  The patch provided explicitly calls ast_stopstream() after a selection is entered.  This appears to be necessary when no other files are streamed following the ast_waitstream() (which is why it does not happen in other places).

The sequence of events which cause this bug to happen are:

1.  (get two people into conference)
2.  Press * (for user or admin menu)
3.  Press 8, 4, or 6 while Allison is reading the menu

The dialplan lines are as follows:

; Test conferencing user (657)
exten => 657,1,Answer
exten => 657,n,MeetMe(,iMs)
exten => 657,n,Hangup

; Test conferencing admin (658)
exten => 658,1,Answer
exten => 658,n,MeetMe(1234,aDiMs)
exten => 658,n,Hangup
Comments:By: Olle Johansson (oej) 2006-02-02 01:38:27.000-0600

Anyone that can repeat this issue? Any more information?

/Housekeeping

By: Mark Spencer (markster) 2006-02-14 12:11:24.000-0600

Fixed in SVN trunk ASTERISK-9690 and 1.2 ASTERISK-9691...  Thanks!

By: Digium Subversion (svnbot) 2008-01-15 16:44:44.000-0600

Repository: asterisk
Revision: 9989

U   trunk/apps/app_meetme.c

------------------------------------------------------------------------
r9989 | markster | 2008-01-15 16:44:44 -0600 (Tue, 15 Jan 2008) | 2 lines

Properly stopsteram on meetme (bug ASTERISK-5980)

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

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

By: Digium Subversion (svnbot) 2008-01-15 16:44:45.000-0600

Repository: asterisk
Revision: 9990

U   branches/1.2/apps/app_meetme.c

------------------------------------------------------------------------
r9990 | markster | 2008-01-15 16:44:45 -0600 (Tue, 15 Jan 2008) | 2 lines

Fix stopstream in menus (bug ASTERISK-5980)

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

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