Re: [SSSD] [PATCH] LDAP: Set default value for dyndns update to false

2013-10-15 Thread Jakub Hrozek
On Tue, Oct 15, 2013 at 10:01:37PM +0200, Jakub Hrozek wrote: > On Tue, Oct 15, 2013 at 02:02:46PM +0200, Pavel Březina wrote: > > On 10/14/2013 02:36 PM, Lukas Slebodnik wrote: > > >ehlo, > > > > > >In some cases, local boolean variable "do_update" could be used > > >without proper initialisation.

Re: [SSSD] [PATCH] LDAP: Set default value for dyndns update to false

2013-10-15 Thread Jakub Hrozek
On Tue, Oct 15, 2013 at 02:02:46PM +0200, Pavel Březina wrote: > On 10/14/2013 02:36 PM, Lukas Slebodnik wrote: > >ehlo, > > > >In some cases, local boolean variable "do_update" could be used > >without proper initialisation. > >Clang static analyser warning: "Assigned value is garbage or undefined

Re: [SSSD] [PATCH] LDAP: Set default value for dyndns update to false

2013-10-15 Thread Pavel Březina
On 10/14/2013 02:36 PM, Lukas Slebodnik wrote: ehlo, In some cases, local boolean variable "do_update" could be used without proper initialisation. Clang static analyser warning: "Assigned value is garbage or undefined" It was not a big problem, because non-zero value for boolean variable mean t

[SSSD] [PATCH] LDAP: Set default value for dyndns update to false

2013-10-14 Thread Lukas Slebodnik
ehlo, In some cases, local boolean variable "do_update" could be used without proper initialisation. Clang static analyser warning: "Assigned value is garbage or undefined" It was not a big problem, because non-zero value for boolean variable mean true. Simple patch is attached. LS >From 0325b79