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

[Kevin Harwell]

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

(Updated April 3, 2015, 5:25 p.m.)


Review request for Asterisk Developers.


Changes
-------

Addressed feedback.


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 (updated)
-----

  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/2353ce7c/attachment-0001.html>