[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/#review5712
-----------------------------------------------------------



trunk/include/asterisk/strings.h
<https://reviewboard.asterisk.org/r/1780/#comment10460>

    This should check to see if there is even a / in the string before uppercasing anything.
    
    Could it ever be given something like:
    custom:state ?



trunk/main/ccss.c
<https://reviewboard.asterisk.org/r/1780/#comment10456>

    finder is now uninitialized.



trunk/main/ccss.c
<https://reviewboard.asterisk.org/r/1780/#comment10457>

    char *devname;
    
    Spacing!



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

    ast_tech_to_upper does not tolerate being handed a NULL pointer.


- rmudgett


On March 1, 2012, 10:42 a.m., opticron wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1780/
> -----------------------------------------------------------
> 
> (Updated March 1, 2012, 10:42 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/20120301/67d5497c/attachment-0001.htm>