[asterisk-dev] [Code Review] Add variable length displayprompt packet to skinny and use octals

opticron reviewboard at asterisk.org
Fri Feb 1 16:33:08 CST 2013


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



/trunk/channels/chan_skinny.c
<https://reviewboard.asterisk.org/r/2294/#comment14783>

    Does SCCP/the phone require a trailing null terminator on this string?  If it does, the length calculation here does not account for it.



/trunk/channels/chan_skinny.c
<https://reviewboard.asterisk.org/r/2294/#comment14784>

    Idem.


- opticron


On Feb. 1, 2013, 4:11 p.m., wedhorn wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2294/
> -----------------------------------------------------------
> 
> (Updated Feb. 1, 2013, 4:11 p.m.)
> 
> 
> Review request for Asterisk Developers and Brad Latus.
> 
> 
> Summary
> -------
> 
> Add new variable length displayprompt packet (0x0145) to skinny. This uses a similar method to the variable length callinfo packet in https://reviewboard.asterisk.org/r/2290/. Uses the new packet if the device is reporting protocol versions >= 17.
> 
> Add the use of octal codes for sending prompts to both the new and old displayprompt messages (also cleaned up soft_key_template_default to use the defined octal codes).
> 
> 
> Diffs
> -----
> 
>   /trunk/channels/chan_skinny.c 380755 
> 
> Diff: https://reviewboard.asterisk.org/r/2294/diff
> 
> 
> Testing
> -------
> 
> Several calls on devices with protocol versions of 6 and 17. Prompts as expected.
> 
> 
> Thanks,
> 
> wedhorn
> 
>

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


More information about the asterisk-dev mailing list