Constant time ObjectClass handling
Matthias Dieter Wallnöfer
mdw at samba.org
Tue Apr 10 07:40:30 MDT 2012
Hi Andrew,
Andrew Bartlett schrieb:
> Thanks. I'll push these at some point over easter, except for:
>
> commit 0dac7b21d97d504653375b8cfddec8a5f35bfc23
> Author: Matthias Dieter Wallnöfer<mdw at samba.org>
> Date: Fri Apr 6 12:40:25 2012 +0200
>
> LDB/s4:dsdb - don't allocate objects on reallocable structures
>
> The "msg->elements" or "struct ldb_message_element" objects (array)
> are subject
> to changes when adding/removing message elements. As far as I
> understand
> "talloc" cannot handle this correctly.
> "msg" however remains always the same for a lifetime of a message.
>
>
> Do you have any evidence that talloc doesn't do this properly? It
> certainly is meant to, so if this isn't working we should investigate
> what is really going on.
Okay, I have not been aware of this circumstance. So I wrote this patch
just in case talloc could not handle this - I was simply not sure about,
let us drop it.
> One questions however: On the key patch:
>
> commit 27078fac8555ef48b8da98650fe944180a177887
> Author: Matthias Dieter Wallnöfer<mdw at samba.org>
> Date: Wed Apr 4 18:40:00 2012 +0200
>
> s4:dsdb - move "objectclass_sort()" out from the objectclass LDB
> module into the schema code
>
> This allows it to be useful for the dbchecker utility in respect to
> object class problems.
> Fix up the API to only work with standardised LDB
> "ldb_message_element"
> structures which do allow much easier interoperations. As a
> consequence this
> leads to some changes in the objectclass module as well.
>
> How did you solve the test failures? When I tried to have it sort
> before doing the get_last_structural_class() and
> check_unrelated_objectclasses() I couldn't make it pass the ldap.py
> python tests.
I am a bit puzzled. But the patches should now be passing ldap.py - as
far as I have tested. Or are you still getting failures?
Thanks,
Matthias
More information about the samba-technical
mailing list