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.
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to