[asterisk-dev] [Code Review] Add IPv6 Address Support To Security Events Framework

elguero reviewboard at asterisk.org
Thu Mar 1 14:04:36 CST 2012


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

(Updated March 1, 2012, 2:04 p.m.)


Review request for Asterisk Developers.


Changes
-------

Removing changes to chan_sip.c since that really is not related to security events.  Will propose this change in another issue.


Summary (updated)
-------

The current API supports only IPv4 for security events. 

*  This patch changes the API to support IPv6 and updates the components that use this API.

*  It also eliminates an error that was being generated since the current implementation was treating an IPv6 socket address as if it was IPv4.

*  Some copyright dates were updated as well.


This addresses bug ASTERISK-19447.
    https://issues.asterisk.org/jira/browse/ASTERISK-19447


Diffs (updated)
-----

  /trunk/CHANGES 357541 
  /trunk/channels/sip/security_events.c 357541 
  /trunk/include/asterisk/security_events_defs.h 357541 
  /trunk/main/manager.c 357541 
  /trunk/main/security_events.c 357541 

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


Testing
-------

Tested SIP implementation on CentOS 5.7, connected with IPv4 clients and IPv6 clients.  Also tested SIP on Fedora 16 vm with IPv4.  Security log now shows IPv4 or IPv6 addresses when a security event is generated.


Thanks,

elguero

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120301/8c820ead/attachment-0001.htm>


More information about the asterisk-dev mailing list