On (30/08/16 16:59), Fabiano Fidêncio wrote: >Lukaš, > >On Tue, Aug 30, 2016 at 4:54 PM, Lukas Slebodnik <lsleb...@redhat.com> wrote: >> ehlo, >> >> Clang static analyzer assume that ldb_search can found >> 0 entries in the tree "cn=sysdb". Thenvariable version >> could be used uninitialized. >> >> We cannot get to such state in sssd but we already handle >> a case for more then one entry. > >I don't think this is the right approach as res->count == 0 seems to >be a valid case for a newly created database (please, correct me if >I'm wrong). > Agree
I should have tried to run unit test before sending a patch LS _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org