[asterisk-dev] [Code Review] Parsing of reg->username containing @domain

Nick Lewis Nick.Lewis at atltelecom.com
Thu Apr 29 05:33:57 CDT 2010


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

(Updated 2010-04-29 05:33:57.592326)


Review request for Asterisk Developers and David Vossel.


Changes
-------

Addresses feedback from dvossel and fixes an uninitialized array element problem in AST_NONSTANDARD_RAW_ARGS


Summary
-------

reg->username is parsed for each registration refresh rather than once on sip reload. A previously committed change that addressed this was reverted because it did not cater for registration strings that contained a peer name in the host field. This change does. 


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


Diffs (updated)
-----

  trunk/channels/chan_sip.c 258227 
  trunk/channels/sip/config_parser.c 258227 
  trunk/channels/sip/include/sip.h 258228 
  trunk/main/app.c 258227 

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


Testing
-------

Passes a new unit test for parsing usernames containing domains. Passes existing unit tests. Passes peername functional test with sip.conf snippet

[general]
register=nick at quirk
[quirk]
fromdomain=10.10.235.48


Thanks,

Nick




More information about the asterisk-dev mailing list