[asterisk-dev] [Code Review] 2633: Filter channels used as internal mechanisms

opticron reviewboard at asterisk.org
Thu Jun 20 11:10:29 CDT 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2633/
-----------------------------------------------------------

(Updated June 20, 2013, 11:10 a.m.)


Status
------

This change has been discarded.


Review request for Asterisk Developers.


Bugs: ASTERISK-21873
    https://issues.asterisk.org/jira/browse/ASTERISK-21873


Repository: Asterisk


Description
-------

This adds a new function pointer to the channel tech which determines whether a channel is fit for external consumption. Examples of channels that would not be fit for external consumption include the announcement and recording channels used by confbridge which are the only two marked as such by this patch.

A lot of this ended up being cleanup to ensure callers of various functions could handle a NULL return.


Diffs
-----

  trunk/apps/confbridge/conf_chan_announce.c 392240 
  trunk/apps/confbridge/conf_chan_record.c 392240 
  trunk/apps/confbridge/confbridge_manager.c 392240 
  trunk/include/asterisk/channel.h 392240 
  trunk/include/asterisk/stasis_channels.h 392240 
  trunk/main/cdr.c 392240 
  trunk/main/cel.c 392241 
  trunk/main/manager_bridging.c 392240 
  trunk/main/manager_channels.c 392240 
  trunk/main/stasis_channels.c 392240 
  trunk/res/parking/parking_manager.c 392240 

Diff: https://reviewboard.asterisk.org/r/2633/diff/


Testing
-------

Ran through CDR unit tests.


Thanks,

opticron

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130620/a7c4082c/attachment-0001.htm>


More information about the asterisk-dev mailing list