[SSSD] [PATCHES] Improve group handling in the PAC responder

Jakub Hrozek jhrozek at redhat.com
Fri Jan 4 15:31:03 UTC 2013


On Fri, Dec 21, 2012 at 12:31:32PM +0100, Sumit Bose wrote:
> On Wed, Dec 19, 2012 at 08:10:02AM -0500, Simo Sorce wrote:
> > On Tue, 2012-12-18 at 21:43 +0100, Sumit Bose wrote:
> > > 
> > > I started to change it but didn't like the resulting code, because it
> > > turned out to be complex and hard to follow. Since the original struct
> > > only user 8 or 12 bytes per group (depending if your are on 32bits or
> > > on
> > > 64bits) I would prefer to keep the old struct for this round of
> > > patches.
> > > 
> > Sorry can't agree, space is only one of the factors, although completely
> > unnecessarily using 2x or 3x number of groups space is ugly enough,
> > But the main issues are really in the code this structure
> > allows/produces.
> > 
> > Still a nack for me.
> 
> The attached patch should be applied on top of the other patches and
> changes the struct as you suggested in your first comment.
> 
> bye,
> Sumit

Why not squash the patch into the patch that introduces the structure?



More information about the sssd-devel mailing list