[asterisk-dev] [Code Review] Support T.38 faxes on local channels

[Russell Bryant]

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



/trunk/channels/chan_local.c
<https://reviewboard.asterisk.org/r/813/#comment5503>

    The DEADLOCK_AVOIDANCE() macro should probably be used here.
    
    Also, you need to re-assign the chan variable after each unlock/lock cycle to make sure it's stall there.


- Russell


On 2010-07-28 12:14:15, Matthew Nicholson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/813/
> -----------------------------------------------------------
> 
> (Updated 2010-07-28 12:14:15)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> This patch enables asterisk to send T.38 faxes over local channels.  To accomplish this, support for ast_channel_queryoption() was added to chan_local and support for passing AST_CONTROL_T38_PARAMETERS control frames through a generic bridge was added.
> 
> 
> Diffs
> -----
> 
>   /trunk/channels/chan_local.c 278500 
>   /trunk/main/channel.c 278500 
> 
> Diff: https://reviewboard.asterisk.org/r/813/diff
> 
> 
> Testing
> -------
> 
> I passed several T.38 faxes over a local channel bridged to a sip channel.
> 
> 
> Thanks,
> 
> Matthew
> 
>