[asterisk-dev] [Code Review]: Fix Dial m and r options generating warnings for voice frames.

[Mark Michelson]

> On March 8, 2012, 8:34 a.m., Matt Jordan wrote:
> > /branches/1.8/apps/app_dial.c, line 832
> > <https://reviewboard.asterisk.org/r/1805/diff/1/?file=25882#file25882line832>
> >
> >     Nice name change.  Looking at wait_for_answer, I can see why you changed the name; however, that's only clear from the context of that method.  Some Doxygen comments explaining what the parameter does might be appropriate in this case.

100% agree on this.


- Mark


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


On March 7, 2012, 4:10 p.m., rmudgett wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1805/
> -----------------------------------------------------------
> 
> (Updated March 7, 2012, 4:10 p.m.)
> 
> 
> Review request for Asterisk Developers and Mark Michelson.
> 
> 
> Summary
> -------
> 
> Incoming calls that do not use ulaw will generate a bunch of warning messages and possibly fail when a DAHDI channel is then called if the Dial m or r options are used.
> 
> You get:
> [Mar 6 20:39:05] WARNING[2994]: chan_dahdi.c:9061 dahdi_write: Cannot handle frames in g722 format
> [Mar 6 20:39:05] WARNING[2994]: app_dial.c:1410 wait_for_answer: Unable to forward voice or dtmf
> 
> A bandaid patch was applied to silence the above warnings for each frame by setting up a translation path for future frames.  The bandaid patch also emits this warning:
> Codec mismatch on channel %s setting write format to %s from %s native formats %s
> 
> This problem also occurs if the dial is forked to multiple destinations: Dial(DAHDI/1&DAHDI/2)
> 
> 
> This addresses bugs ASTERISK-16901 and ASTERISK-17541.
>     https://issues.asterisk.org/jira/browse/ASTERISK-16901
>     https://issues.asterisk.org/jira/browse/ASTERISK-17541
> 
> 
> Diffs
> -----
> 
>   /branches/1.8/apps/app_dial.c 358576 
>   /branches/1.8/main/channel.c 358576 
> 
> Diff: https://reviewboard.asterisk.org/r/1805/diff
> 
> 
> Testing
> -------
> 
> Without patch, the codecs are not setup when Dial options m or r are used and the SIP codec is not ulaw.
> 
> With the patch, the voice frames are not passed and the diagnostic messages are not generated.
> 
> 
> Thanks,
> 
> rmudgett
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120308/46a7f237/attachment.htm>