[PATCH] Fix dn+binary linked attribute behavior (bug 11139)

Douglas Bagnall douglas.bagnall at catalyst.net.nz
Thu Jan 5 04:05:31 UTC 2017


hi Bob,

I see a couple of superficial issues.
>  
> +import os

You don't actually use this.

> @@ -37,12 +40,11 @@ from ldb import (
>      Message,
>      FLAG_MOD_ADD,
>      FLAG_MOD_REPLACE,
> +    FLAG_MOD_DELETE
>      )

or this.

> +
> +        link_end_dn = ldb.Dn(self.ldb_dc1, "OU=test_ou,%s" % str(self.ldb_dc1.get_default_basedn()))

here the line length is getting a bit out of control and the str() is
redundant (because "%s" implies str()). There are other instances of that.

> +
> +        self._net_drs_replicate(DC=self.dnsname_dc2, fromDC=self.dnsname_dc1, nc_dn=self.schema_dn, forced=True)
> +        self._net_drs_replicate(DC=self.dnsname_dc2, fromDC=self.dnsname_dc1, nc_dn=self.domain_dn, forced=True)
> +

and line length again.

> From 0a1a65c99e6b8ec39ad0c7018e05759343899e51 Mon Sep 17 00:00:00 2001
> From: Andrew Bartlett <abartlet at samba.org>
> Date: Mon, 12 Oct 2015 15:51:37 +1300
> Subject: [PATCH 2/2] repl_meta_data: Remove the correct linked attribute
>  forward link

This one looks good.

cheers,
Douglas




More information about the samba-technical mailing list