[asterisk-dev] [Code Review] 4582: res_pjsip: config option 'timers' can't be set to 'no'

[rmudgett]

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

Ship it!


Ship It!

- rmudgett


On April 3, 2015, 5:52 p.m., Kevin Harwell wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/4582/
> -----------------------------------------------------------
> 
> (Updated April 3, 2015, 5:52 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-24910
>     https://issues.asterisk.org/jira/browse/ASTERISK-24910
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> When setting the configuration option 'timers' equal to 'no' the bit flag was not properly negated. This patch clears all associated flags and only sets the specified one. pjsip will handle any necessary flag combinations. Also went ahead and did similar for the '100rel' option.
> 
> 
> Diffs
> -----
> 
>   branches/13/res/res_pjsip/pjsip_configuration.c 433966 
>   branches/13/res/res_pjsip.c 433966 
> 
> Diff: https://reviewboard.asterisk.org/r/4582/diff/
> 
> 
> Testing
> -------
> 
> Made sure the option can now be set to 'no' and that it clears the bit. Also set it to the other values and reloaded to make sure the field was updated correctly.
> 
> 
> Thanks,
> 
> Kevin Harwell
> 
>

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