[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-09-01 Thread jhrozek
jhrozek commented on a pull request """ * master: 5bd3bef4a655fdfacd2f5df8a2343fe7bc68a771 """ See the full comment at https://github.com/SSSD/sssd/pull/7#issuecomment-244058146 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-09-01 Thread jhrozek
jhrozek commented on a pull request """ On Wed, Aug 31, 2016 at 07:15:10AM -0700, lslebodn wrote: > On (31/08/16 01:47), sumit-bose wrote: > >On Wed, Aug 31, 2016 at 01:30:12AM -0700, Jakub Hrozek wrote: > >> On Wed, Aug 31, 2016 at 12:36:37AM -0700, sumit-bose wrote: > >> > On Tue, Aug 30, 2016

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-08-31 Thread lslebodn
lslebodn commented on a pull request """ On (31/08/16 01:47), sumit-bose wrote: >On Wed, Aug 31, 2016 at 01:30:12AM -0700, Jakub Hrozek wrote: >> On Wed, Aug 31, 2016 at 12:36:37AM -0700, sumit-bose wrote: >> > On Tue, Aug 30, 2016 at 12:36:20PM -0700, Jakub Hrozek wrote: >> > > On Tue, Aug 30,

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-08-31 Thread sumit-bose
sumit-bose commented on a pull request """ On Wed, Aug 31, 2016 at 01:30:12AM -0700, Jakub Hrozek wrote: > On Wed, Aug 31, 2016 at 12:36:37AM -0700, sumit-bose wrote: > > On Tue, Aug 30, 2016 at 12:36:20PM -0700, Jakub Hrozek wrote: > > > On Tue, Aug 30, 2016 at 11:47:09AM -0700, lslebodn wrote:

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-08-31 Thread jhrozek
jhrozek commented on a pull request """ On Wed, Aug 31, 2016 at 12:36:37AM -0700, sumit-bose wrote: > On Tue, Aug 30, 2016 at 12:36:20PM -0700, Jakub Hrozek wrote: > > On Tue, Aug 30, 2016 at 11:47:09AM -0700, lslebodn wrote: > > > > About the discussion I saw on #sssd in backscroll, the

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-08-31 Thread sumit-bose
sumit-bose commented on a pull request """ On Tue, Aug 30, 2016 at 12:36:20PM -0700, Jakub Hrozek wrote: > On Tue, Aug 30, 2016 at 11:47:09AM -0700, lslebodn wrote: > > About the discussion I saw on #sssd in backscroll, the rfc2307bis schema > only uses the member attribute because IIRC the RFC

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-08-30 Thread jhrozek
jhrozek commented on a pull request """ OK, also looking at gdb confirms this is the right fix of course: (gdb) frame 0 #0 sdap_initgr_nested_get_membership_diff (mem_ctx=0x1176190, sysdb=0x1120040, opts=0x1134d90, dom=0x110b610, group=0x1184b80, all_groups=0x1184a80, groups_count=5,

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-08-30 Thread jhrozek
jhrozek commented on a pull request """ On Tue, Aug 30, 2016 at 11:47:09AM -0700, lslebodn wrote: > Please provide a test-case (probably a hierarchy of groups) I was able to reproduce with: $ ipa group-show group20 Group name: group20 GID: 935600011 Member groups: group10, group11

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-08-30 Thread lslebodn
lslebodn commented on a pull request """ Please provide a test-case (probably a hierarchy of groups) """ See the full comment at https://github.com/SSSD/sssd/pull/7#issuecomment-243540136 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org