Problem with AD membership in an AD with more the 100.000 group (possible regression in 4.12?)
abartlet at samba.org
Fri May 21 09:31:30 UTC 2021
On Thu, 2021-05-20 at 23:29 +1200, Andrew Bartlett wrote:
> On Thu, 2021-05-20 at 19:46 +1200, Andrew Bartlett via samba-
> > On Thu, 2021-05-20 at 08:36 +0200, Dr. Hansjörg Maurer wrote:
> > > Am 19.05.21 um 11:55 schrieb Andrew Bartlett:
> > > > On Wed, 2021-05-19 at 07:56 +0200, Dr. Hansjörg Maurer wrote:
> > > > > > OK, so this case it doesn't happen, which is the one I
> > > > > > patched.
> > > > > the gdb run took place with the unpatched version
> > > > >
> > > > Thanks, can you try with the patched one? I'm thinking we must
> > > > be
> > > > hitting a different case now, if you still have issues, as the
> > > > lines
> > > > you show are gone with this patch (I hope).
> > > Hi Andrew
> > >
> > > here is the gdb bt output from the patched version
> > >
> > > I patched the sources an rebuild the RPM with the spec file.
> > > I would expect, this would apply the changes to the winbind.idl
> > > file?
> > Thanks, yes this seems to have been applied. I've looked at this
> > some
> > more and I see where we leak the tokens.
> > I'll see what we can do, I think for a conformant array (what I've
> > changed this to) we can be more efficient.
> > Andrew Bartlett
> Try the attached patch. I think I've finally fixed the issue.
> See https://gitlab.com/samba-team/samba/-/merge_requests/1972
Just be aware that this isn't a working patch (see the failed
I'm not sure if I'll be able to get this approach to work, sadly.
Andrew Bartlett (he/him) https://samba.org/~abartlet/
Samba Team Member (since 2001) https://samba.org
Samba Team Lead, Catalyst IT https://catalyst.net.nz/services/samba
Samba Development and Support, Catalyst IT - Expert Open Source
More information about the samba-technical