[asterisk-dev] [Code Review] 4075: parking/tests: Running res_parking unit tests would cause assertions and possibly a crash due to attempting to play MOH on a channel with no formats

[rmudgett]

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



/branches/13/res/parking/parking_tests.c
<https://reviewboard.asterisk.org/r/4075/#comment24029>

    You don't need to use safe_channel_release() here.  You can just use ast_channel_release() since you know that alice is not NULL.


- rmudgett


On Oct. 13, 2014, 5:18 p.m., Jonathan Rose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/4075/
> -----------------------------------------------------------
> 
> (Updated Oct. 13, 2014, 5:18 p.m.)
> 
> 
> Review request for Asterisk Developers and Matt Jordan.
> 
> 
> Bugs: ASTERISK-24413
>     https://issues.asterisk.org/jira/browse/ASTERISK-24413
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> This patch simply follows the suggested fix of specifying the format for test channels in the same manner as was done for the CDR unit tests.
> 
> 
> Diffs
> -----
> 
>   /branches/13/res/parking/parking_tests.c 425404 
> 
> Diff: https://reviewboard.asterisk.org/r/4075/diff/
> 
> 
> Testing
> -------
> 
> Ran tests prior to patch and got assertions.  Assertions no longer occurred with the patch in place.
> 
> 
> Thanks,
> 
> Jonathan Rose
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20141013/d8742fdb/attachment.html>