Fix incorrect msDS-IntID handling
abartlet at samba.org
Tue Jan 5 07:54:37 UTC 2016
On Mon, 2016-01-04 at 11:53 +0100, Stefan Metzmacher wrote:
> Hi Andrew,
> > > I still need to resolve some issues in the test-suite (probably
> > > drop
> > > one aspect of it, as I can't make it deterministic), but in case
> > > you
> > > are intersted please see the attached, plus the re-upload of the
> > > DB
> > > in
> > >
> > > git://git.catalyst.net.nz/samba.git replMetaData-attid
> > >
> > > http://git.catalyst.net.nz/gitweb?p=samba.git;a=shortlog;h=refs/h
> > > eads
> > > /r
> > > eplMetaData-attid
> > A final version of these patches is in that repo.
> I'm ignoring the python related patches, which are currently on
> > Please review/push!
> in err_replmetadata_incorrect_attid()
> + if o.local_usn > hash_att[att].local_usn:
> + # This is always what we would have sent over
> + # because the DRS server will have sent the
> + # msDS-IntID, but with the values from both
> + # attribute entries.
> + hash_att[att].version = o.version
> + hash_att[att].originating_change_time =
> + hash_att[att].originating_invocation_id =
> + hash_att[att].originating_usn =
> + hash_att[att].local_usn = o.local_usn
> Should we have a "continue" here
> otherwise it makes no sense if we always do "hash_att[att] = o"
> + hash_att[att] = o
> + set_att.add(att.lower())
Well spotted. Thanks
> In general are the source4/selftest/provisions/ changes required to
> in the "dbcheck: Fix incorrect/duplicate attrid in
It is the test for that change.
> Or can they be part of the "selftest: Update release-4-1-0rc3 with
> test records"
I don't want anything extra in that commit, it is too large as it is
and essentially impossible to review. Like we do ABI updates in their
own commit, this belongs in it's own commit.
Andrew Bartlett http://samba.org/~abartlet/
Authentication Developer, Samba Team http://samba.org
Samba Developer, Catalyst IT http://catalyst.net.nz/services/samba
More information about the samba-technical