s4-provision: Fixed typos and redundant code
authorEndi S. Dewata <edewata@redhat.com>
Tue, 19 Jan 2010 03:01:08 +0000 (21:01 -0600)
committerMatthias Dieter Wallnöfer <mwallnoefer@yahoo.de>
Sat, 23 Jan 2010 21:41:10 +0000 (22:41 +0100)
Signed-off-by: Matthias Dieter Wallnöfer <mwallnoefer@yahoo.de>
source4/scripting/python/samba/provisionbackend.py

index ae58a163178cb103bfe560efce6246bfd7a775b5..8241ed69ff851418b4851283db20fecc126e17ed 100644 (file)
@@ -704,7 +704,7 @@ class FDSBackend(LDAPBackend):
             os.path.join(self.paths.ldapdir, "slapd-samba4", "ldif2db"), "-s", self.sambadn, "-i", self.samba_ldif],
             close_fds=True, shell=False)
         if retcode != 0:
-            raise("ldif2db failed")
+            raise ProvisioningError("ldif2db failed")
 
     def post_setup(self):
         ldapi_db = Ldb(self.ldapi_uri, credentials=self.credentials)
@@ -717,16 +717,16 @@ class FDSBackend(LDAPBackend):
             dn = str(res[i]["dn"])
             ldapi_db.delete(dn)
             
-            aci = """(targetattr = "*") (version 3.0;acl "full access to all by samba-admin";allow (all)(userdn = "ldap:///CN=samba-admin,%s");)""" % self.sambadn
+        aci = """(targetattr = "*") (version 3.0;acl "full access to all by samba-admin";allow (all)(userdn = "ldap:///CN=samba-admin,%s");)""" % self.sambadn
         
-            m = ldb.Message()
-            m["aci"] = ldb.MessageElement([aci], ldb.FLAG_MOD_REPLACE, "aci")
+        m = ldb.Message()
+        m["aci"] = ldb.MessageElement([aci], ldb.FLAG_MOD_REPLACE, "aci")
 
-            m.dn = ldb.Dn(1, self.names.domaindn)
-            ldapi_db.modify(m)
+        m.dn = ldb.Dn(ldapi_db, self.names.domaindn)
+        ldapi_db.modify(m)
             
-            m.dn = ldb.Dn(1, self.names.configdn)
-            ldapi_db.modify(m)
+        m.dn = ldb.Dn(ldapi_db, self.names.configdn)
+        ldapi_db.modify(m)
             
-            m.dn = ldb.Dn(1, self.names.schemadn)
-            ldapi_db.modify(m)
+        m.dn = ldb.Dn(ldapi_db, self.names.schemadn)
+        ldapi_db.modify(m)