[SSSD] Re: [PATCH] SDAP: Fix settig paging attribute in sdap_get_generic_ext_send

2016-09-13 Thread Lukas Slebodnik
On (09/09/16 13:09), Sumit Bose wrote: >On Fri, Sep 09, 2016 at 12:48:47PM +0200, Lukas Slebodnik wrote: >> On (31/08/16 07:45), Fabiano FidĂȘncio wrote: >> >On Tue, Aug 30, 2016 at 5:26 PM, Lukas Slebodnik >> >wrote: >> >> ehlo, >> >> >> >> We should set pagging flag in

[SSSD] Re: [PATCH] SDAP: Fix settig paging attribute in sdap_get_generic_ext_send

2016-09-09 Thread Sumit Bose
On Fri, Sep 09, 2016 at 12:48:47PM +0200, Lukas Slebodnik wrote: > On (31/08/16 07:45), Fabiano FidĂȘncio wrote: > >On Tue, Aug 30, 2016 at 5:26 PM, Lukas Slebodnik wrote: > >> ehlo, > >> > >> We should set pagging flag in state and not in local > >> variable which is not read

[SSSD] Re: [PATCH] SDAP: Fix settig paging attribute in sdap_get_generic_ext_send

2016-08-30 Thread Fabiano FidĂȘncio
On Tue, Aug 30, 2016 at 5:26 PM, Lukas Slebodnik wrote: > ehlo, > > We should set pagging flag in state and not in local > variable which is not read anywhere in the function. > > Found by clang static analyzer. > > Do we need this patch also to stable branch? > > LS > >