[asterisk-dev] [Code Review] Fix case-sensitivity for device-specific event subscriptions and CCSS

opticron reviewboard at asterisk.org
Fri Mar 2 10:38:58 CST 2012


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

(Updated March 2, 2012, 10:38 a.m.)


Review request for Asterisk Developers.


Changes
-------

Comments addressed.


Summary
-------

This change fixes case-sensitivity for device-specific subscriptions such that the technology identifier is case-insensitive while the remainder of the device string is still case-sensitive.  This should also preserve the original case of the device string as passed in to the event system.  CCSS is the only feature affected as it is the only consumer of device-specific event subscriptions.

The second part of this patch addresses similar case-sensitivity issues within CCSS itself that prevented it from functioning correctly after the fix to the events system.


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


Diffs (updated)
-----

  trunk/include/asterisk/strings.h 357683 
  trunk/main/ccss.c 357683 
  trunk/main/event.c 357683 
  trunk/tests/test_event.c 357683 

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


Testing
-------

Added unit tests for the event system changes and used the pending CCSS test to verify that CCSS functioned correctly when cases were mixed.


Thanks,

opticron

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120302/d3a5097a/attachment.htm>


More information about the asterisk-dev mailing list