[asterisk-dev] [Code Review] 2857: bridge_softmix: Add a warning when no timing source is available

Olle E Johansson reviewboard at asterisk.org
Sun Sep 15 02:21:41 CDT 2013


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


Without understanding the source code, I just have to say that I really support adding warnings like this, that make it easier to manager and understand Asterisk.

- Olle E Johansson


On Sept. 15, 2013, 3:22 a.m., Matt Jordan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2857/
> -----------------------------------------------------------
> 
> (Updated Sept. 15, 2013, 3:22 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> While tinkering with ConfBridge on an Asterisk setup with a limited set of modules installed, I ran into an error reported from ConfBridge that it couldn't create a bridge due to a failure to create a softmix bridge. Unfortunately, bridge_softmix itself didn't relay why it wasn't created.
> 
> Inspecting the code revealed there are three ways creation of a softmix bridge can fail:
>  1. Memory allocation failure
>  2. NULL returned from ast_timer_open
>  3. Thread creation failure
> 
> 1 and 3 are both unlikely and likely to output an error from their respective routines. Number 2, however, is silent if no timing module is loaded.
> 
> This patch adds a warning on that off nominal path.
> 
> A lot of folks run Asterisk without a timing source (for whatever reason), which means - since softmix bridges will be used a lot more and in more places in 12 than in the past - a WARNING here is pretty useful.
> 
> 
> Diffs
> -----
> 
>   /branches/11/bridges/bridge_softmix.c 398805 
> 
> Diff: https://reviewboard.asterisk.org/r/2857/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Matt Jordan
> 
>

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


More information about the asterisk-dev mailing list