[SSSD] [PATCHES] IPA: fixed for initgroups with overrides

Jakub Hrozek jhrozek at redhat.com
Tue May 5 13:22:14 UTC 2015


On Thu, Apr 30, 2015 at 04:50:44PM +0200, Jakub Hrozek wrote:
> On Wed, Apr 29, 2015 at 05:34:29PM +0200, Sumit Bose wrote:
> > +static errno_t ipa_get_ad_ipa_membership_step(struct tevent_req *req)
> > +{
> > +    struct ipa_get_ad_acct_state *state = tevent_req_data(req,
> > +                                                struct ipa_get_ad_acct_state);
> > +    errno_t ret;
> 
> Sorry, there's one more warning, the 'ret' here is unused.
> 
> > +    struct tevent_req *subreq;
> >  
> >      /* For initgroups request we have to check IPA group memberships of AD
> >       * users. This has to be done for other user-request as well to make sure
> > -- 
> > 2.1.0
> > 
> 
> > _______________________________________________
> > sssd-devel mailing list
> > sssd-devel at lists.fedorahosted.org
> > https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

I only found an unrelated issue with the default trust view and group
members in my testing. Our Red Hat QE colleagues also confirmed that the
patches don't break anything, so I'll ACK them and fix the unused ret
before pushing.



More information about the sssd-devel mailing list