[SSSD] [PATCH] Consolidation of functions that make realm upper-case

Ondrej Kos okos at redhat.com
Tue Aug 21 14:48:08 UTC 2012


Fixed

Function was modified to just return the given string as uppercase. The 
part where the whole principal is being processed was restored to it's 
previous state.

New patch is attached

Ondrej

On 08/21/2012 04:21 PM, Jakub Hrozek wrote:
> On Tue, Aug 21, 2012 at 04:11:19PM +0200, Ondrej Kos wrote:
>> https://fedorahosted.org/sssd/ticket/1491
>>
>> I prepared function *get_uppercase_realm* accessible through
>> src/util/util.h which returns uppercase realm string. If whole
>> principal is passed, it transforms only the part after '@'
>> character. Multiple uses of *toupper* have been replaced and
>> modified to use mentioned new function.
>>
>> Ondrej
>
> I nacked the patch off-list, test statements were left in and the code
> violated style guidelines.
> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
>


-- 
Ondrej Kos
Associate Software Engineer
Identity Management
Red Hat Czech

cell:  +420-736-417-909
phone: +420-532-294-558
ext.:  82-62558
irc:   okos @ #brno
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Consolidation-of-functions-that-make-realm-upper-cas.patch
Type: text/x-patch
Size: 4962 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20120821/cd45bf2b/attachment.bin>


More information about the sssd-devel mailing list