From: Matthias Dieter Wallnöfer <mwallnoefer@yahoo.de>
Date: Mon, 17 Aug 2009 09:47:53 +0000 (+0200)
Subject: s4:python tools - Centralise the lookups for the default domain (root) in the call... 
X-Git-Url: http://git.samba.org/?p=metze%2Fsamba%2Fwip.git;a=commitdiff_plain;h=0a5ea25d21a9745508e118a20fca837e74b031df

s4:python tools - Centralise the lookups for the default domain (root) in the call "domain_dn" from SamDB
---

diff --git a/source4/setup/enableaccount b/source4/setup/enableaccount
index 1b7341946f9c..d4e954074b13 100755
--- a/source4/setup/enableaccount
+++ b/source4/setup/enableaccount
@@ -56,14 +56,9 @@ else:
 samdb = SamDB(url=url, session_info=system_session(), 
               credentials=creds, lp=lp)
 
-if opts.base is None:
-	res = samdb.search("", scope=ldb.SCOPE_BASE, 
-			  expression="(defaultNamingContext=*)", 
-			  attrs=["defaultNamingContext"])
-	assert(len(res) == 1 and res[0]["defaultNamingContext"] is not None)
-	domain_dn = res[0]["defaultNamingContext"][0]
-else:
-	domain_dn = opts.base
+domain_dn = opts.base
+if domain_dn is None:
+        domain_dn = SamDB.domain_dn(samdb)
 
 filter = "(&(objectClass=user)(samAccountName=%s))" % username
 
diff --git a/source4/setup/pwsettings b/source4/setup/pwsettings
index a2708531a156..fc59f8c4041c 100755
--- a/source4/setup/pwsettings
+++ b/source4/setup/pwsettings
@@ -64,14 +64,10 @@ else:
 samdb = SamDB(url=url, session_info=system_session(),
               credentials=creds, lp=lp)
 
-res = samdb.search("", scope=ldb.SCOPE_BASE,
-	  expression="(defaultNamingContext=*)",
-	  attrs=["defaultNamingContext"])
-assert(len(res) == 1 and res[0]["defaultNamingContext"] is not None)
-domain_dn = res[0]["defaultNamingContext"][0]
-
-res = samdb.search(domain_dn, scope=ldb.SCOPE_BASE, attrs=["pwdProperties",
-  "pwdHistoryLength", "minPwdLength", "minPwdAge", "maxPwdAge"])
+domain_dn = SamDB.domain_dn(samdb)
+res = samdb.search(domain_dn, scope=ldb.SCOPE_BASE,
+  attrs=["pwdProperties", "pwdHistoryLength", "minPwdLength", "minPwdAge",
+  "maxPwdAge"])
 assert(len(res) == 1)
 try:
 	pwd_props = int(res[0]["pwdProperties"][0])