[asterisk-commits] jpeeler: branch 1.6.2 r273356 - in /branches/1.6.2: ./ apps/app_meetme.c

[SVN commits to the Asterisk project]

Author: jpeeler
Date: Thu Jul  1 10:14:41 2010
New Revision: 273356

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=273356
Log:
Merged revisions 273355 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
  r273355 | jpeeler | 2010-07-01 10:12:31 -0500 (Thu, 01 Jul 2010) | 19 lines
  
  Merged revisions 273354 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r273354 | jpeeler | 2010-07-01 10:05:43 -0500 (Thu, 01 Jul 2010) | 12 lines
    
    Ensure channel placed in meetme in ringing state is properly hung up.
    
    An outgoing channel placed in meetme while still ringing which was then hung up
    would not exit meetme and the channel was not properly destroyed. Specifically
    checking for this scenario by looking at the appropriate control frames resolves
    the issue.
    
    (closes issue #15871)
    Reported by: Ivan
    Patches: 
          meetme_congestion_trunk_v2.patch uploaded by Ivan (license 229)
  ........
................

Modified:
    branches/1.6.2/   (props changed)
    branches/1.6.2/apps/app_meetme.c

Propchange: branches/1.6.2/
------------------------------------------------------------------------------
Binary property 'trunk-merged' - no diff available.

Modified: branches/1.6.2/apps/app_meetme.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.6.2/apps/app_meetme.c?view=diff&rev=273356&r1=273355&r2=273356
==============================================================================
--- branches/1.6.2/apps/app_meetme.c (original)
+++ branches/1.6.2/apps/app_meetme.c Thu Jul  1 10:14:41 2010
@@ -3170,6 +3170,18 @@
 					}
 				} else if (f->frametype == AST_FRAME_NULL) {
 					/* Ignore NULL frames. It is perfectly normal to get these if the person is muted. */
+				} else if (f->frametype == AST_FRAME_CONTROL) {
+					switch (f->subclass) {
+					case AST_CONTROL_BUSY:
+					case AST_CONTROL_CONGESTION:
+						ast_frfree(f);
+						goto outrun;
+						break;
+					default:
+						ast_debug(1, 
+							"Got ignored control frame on channel %s, f->frametype=%d,f->subclass=%d\n",
+							chan->name, f->frametype, f->subclass);
+					}
 				} else {
 					ast_debug(1, 
 						"Got unrecognized frame on channel %s, f->frametype=%d,f->subclass=%d\n",