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

Kevin Harwell reviewboard at asterisk.org
Mon Apr 6 14:23:30 CDT 2015


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

(Updated April 6, 2015, 2:23 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 434131


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/20150406/46ebc957/attachment.html>


More information about the asterisk-dev mailing list