[SSSD] [PATCH] Null pointer dereferenced.

Pavel Březina pbrezina at redhat.com
Thu Nov 29 12:02:27 UTC 2012


On 11/28/2012 01:47 PM, Jakub Hrozek wrote:
> On Tue, Nov 27, 2012 at 11:42:02AM -0500, Stephen Gallagher wrote:
>> Actually, I agree with Michal here. Performing the NULL check first
>> reads better (and is more in line with our coding style elsewhere in
>> SSSD). The patch's real changes are viewable with 'git diff -w
>> HEAD^' after applying it, where it's clear that he's only fixed up
>> three debug messages and added the NULL check before the
>> previously-indented section.
>>
>> I'll ack this patch as-is. There is no other appropriate split for it.
>
> I agree. I like the code flow.
>
> I also tested sysdb upgrade from 0.12 to 0.14 (aka from 6.3 to 6.4) and
> it worked fine.
>
> Ack

This was pushed to master and sssd-1-9.




More information about the sssd-devel mailing list