[asterisk-addons-commits] russell: branch 1.4 r414 - /branches/1.4/asterisk-ooh323c/src/chan_h323.c

[SVN commits to the Asterisk addons project]

Author: russell
Date: Thu Jul 19 10:12:54 2007
New Revision: 414

URL: http://svn.digium.com/view/asterisk-addons?view=rev&rev=414
Log:
fix HOLD and UNHOLD control frame processing to the ooh323 channel driver.
The minimum thing that every channel driver should do with these is start
or stop music on hold on the channel.  Optionally, depending on the channel,
it may make sense to add configuration options to pass this information down
so that the remote end can handle the HOLD signalling in the way that they
choose.  For example, chan_iax2 will pass this signalling across the network,
and chan_zap will pass this signalling on a PRI.
(This fixes a problem I saw on the asterisk-users mailing list)

Modified:
    branches/1.4/asterisk-ooh323c/src/chan_h323.c

Modified: branches/1.4/asterisk-ooh323c/src/chan_h323.c
URL: http://svn.digium.com/view/asterisk-addons/branches/1.4/asterisk-ooh323c/src/chan_h323.c?view=diff&rev=414&r1=413&r2=414
==============================================================================
--- branches/1.4/asterisk-ooh323c/src/chan_h323.c (original)
+++ branches/1.4/asterisk-ooh323c/src/chan_h323.c Thu Jul 19 10:12:54 2007
@@ -1036,9 +1036,16 @@
             ast_set_flag(p, H323_ALREADYGONE);
          }
          break;
+      case AST_CONTROL_HOLD:
+         ast_moh_start(ast, data, NULL);      
+         break;
+      case AST_CONTROL_UNHOLD:
+	     ast_moh_stop(ast);
+         break;
       case AST_CONTROL_PROCEEDING:
       case AST_CONTROL_RINGING:
       case AST_CONTROL_PROGRESS:
+      case -1;
          break;
       default:
          ast_log(LOG_WARNING,"Don't know how to indicate condition %d on %s\n",