[asterisk-commits] alecdavis: trunk r249801 - /trunk/apps/app_echo.c

[SVN commits to the Asterisk project]

Author: alecdavis
Date: Tue Mar  2 01:38:56 2010
New Revision: 249801

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=249801
Log:
fixes ability to exit echo app

when called from a ISDN channel, null frames prevent '#' exit.
Now only echo back VOICE and DTMF frames

(issue #16880)
Reported by: alecdavis
Patches: 
      echo_exit.diff.txt uploaded by alecdavis (license 585)
Tested by: alecdavis


Modified:
    trunk/apps/app_echo.c

Modified: trunk/apps/app_echo.c
URL: http://svnview.digium.com/svn/asterisk/trunk/apps/app_echo.c?view=diff&rev=249801&r1=249800&r2=249801
==============================================================================
--- trunk/apps/app_echo.c (original)
+++ trunk/apps/app_echo.c Tue Mar  2 01:38:56 2010
@@ -62,17 +62,26 @@
 		if (!f) {
 			break;
 		}
-		f->delivery.tv_sec = 0;
-		f->delivery.tv_usec = 0;
-		if (ast_write(chan, f)) {
-			ast_frfree(f);
-			goto end;
-		}
-		if ((f->frametype == AST_FRAME_DTMF) && (f->subclass.integer == '#')) {
-			res = 0;
-			ast_frfree(f);
-			goto end;
-		}
+		switch (f->frametype) {
+		case AST_FRAME_VOICE:
+		case AST_FRAME_DTMF:
+			f->delivery.tv_sec = 0;
+			f->delivery.tv_usec = 0;
+			if (ast_write(chan, f)) {
+				ast_frfree(f);
+				goto end;
+			}
+			if (f->frametype == AST_FRAME_DTMF) {
+				if (f->subclass.integer == '#') {
+					res = 0;
+					ast_frfree(f);
+					goto end;
+				}
+			}
+			break;
+		default:
+			break;
+		}		
 		ast_frfree(f);
 	}
 end: