[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

Jonathan Rose reviewboard at asterisk.org
Mon Oct 13 17:05:40 CDT 2014



> On Oct. 13, 2014, 4:58 p.m., rmudgett wrote:
> > /branches/13/res/parking/parking_tests.c, lines 60-64
> > <https://reviewboard.asterisk.org/r/4075/diff/1/?file=68013#file68013line60>
> >
> >     Why is chan in parentheses?

Because I lazily copied this function from a macro and overlooked all of these things.


- Jonathan


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


On Oct. 13, 2014, 3:59 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, 3:59 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/815b3730/attachment.html>


More information about the asterisk-dev mailing list