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

Jakub Hrozek jhrozek at redhat.com
Wed Aug 22 13:18:05 UTC 2012


On Tue, Aug 21, 2012 at 04:48:08PM +0200, Ondrej Kos wrote:
> 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
> 

Fix the braces in this hunk:
> +
> +    realm = talloc_strdup(memctx, name);
> +    if (!realm)
> +    {
> +        return NULL;
> +    }

and I'll ack :-)

See http://www.freeipa.org/page/Coding_Style#Conditions_and_Statements
for the full guidelines.



More information about the sssd-devel mailing list