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

[rmudgett]

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

Ship it!


Other than the optional performance suggestions below, this looks good to go.



trunk/main/event.c
<https://reviewboard.asterisk.org/r/1780/#comment10476>

    Doing the ast_strdupa is only needed if it is an AST_EVENT_IE_DEVICE.
    
    This is only a performance enhancement.



trunk/main/event.c
<https://reviewboard.asterisk.org/r/1780/#comment10477>

    Doing the ast_strdupa is only needed if it is an AST_EVENT_IE_DEVICE.
    
    This is only a performance enhancement.


- rmudgett


On March 2, 2012, 10:38 a.m., opticron wrote:
> 
> -----------------------------------------------------------
> 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.
> 
> 
> 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
> -----
> 
>   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/e783047c/attachment-0001.htm>