[asterisk-dev] [Code Review] SIP from-header is parsed twice

Nick Lewis Nick.Lewis at atltelecom.com
Thu Jun 10 11:31:22 CDT 2010


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

Review request for Asterisk Developers and David Vossel.


Summary
-------

The handlers in chan_sip.c for invite and subscription requests parse the from-header twice. The first time is in check_user_full and the second time is in get_destination. This change removes the parsing from these functions and does it beforehand (with an abnf compliant parser)


This addresses bug 16897.
    https://issues.asterisk.org/view.php?id=16897


Diffs
-----

  trunk/channels/chan_sip.c 268968 
  trunk/channels/sip/reqresp_parser.c 268968 

Diff: https://reviewboard.asterisk.org/r/701/diff


Testing
-------

Compiles, runs, passes associated unit test. CID field appears correct in sip show channel 


Thanks,

Nick




More information about the asterisk-dev mailing list