From 8be99413540f0a9f1d4a6b6630fcfd5f1e3baefc Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Fri, 9 Jan 2009 18:18:28 +1100 Subject: Don't parse the schema data twice --- source4/scripting/python/samba/provision.py | 20 ++++++++------------ 1 file changed, 8 insertions(+), 12 deletions(-) (limited to 'source4') diff --git a/source4/scripting/python/samba/provision.py b/source4/scripting/python/samba/provision.py index 4da8079e09..177f0c3639 100644 --- a/source4/scripting/python/samba/provision.py +++ b/source4/scripting/python/samba/provision.py @@ -784,10 +784,9 @@ def setup_samdb(path, setup_path, session_info, credentials, lp, if serverrole == "domain controller": samdb.set_invocation_id(invocationid) - load_schema(setup_path, samdb, names.schemadn, names.netbiosname, - names.configdn, names.sitename, names.serverdn, - names.hostname) - + schema_data = load_schema(setup_path, samdb, names.schemadn, names.netbiosname, + names.configdn, names.sitename, names.serverdn, + names.hostname) samdb.transaction_start() try: @@ -852,13 +851,8 @@ def setup_samdb(path, setup_path, session_info, credentials, lp, "PREFIXMAP_B64": b64encode(prefixmap) }) - message("Setting up sam.ldb Samba4 schema") - setup_add_ldif(samdb, setup_path("schema_samba4.ldif"), - {"SCHEMADN": names.schemadn }) - - message("Setting up sam.ldb AD schema") - data = get_schema_data(setup_path, {"SCHEMADN": names.schemadn}) - samdb.add_ldif(data) + message("Setting up sam.ldb schema") + samdb.add_ldif(schema_data) setup_add_ldif(samdb, setup_path("aggregate_schema.ldif"), {"SCHEMADN": names.schemadn}) @@ -1661,6 +1655,8 @@ def load_schema(setup_path, samdb, schemadn, netbiosname, configdn, sitename, :param configdn: DN of the configuration :param serverdn: DN of the server :param servername: Host name of the server + + Returns the schema data loaded, to avoid double-parsing when then needing to add it to the db """ schema_data = get_schema_data(setup_path, {"SCHEMADN": schemadn}) schema_data += open(setup_path("schema_samba4.ldif"), 'r').read() @@ -1681,7 +1677,7 @@ def load_schema(setup_path, samdb, schemadn, netbiosname, configdn, sitename, }) check_all_substituted(head_data) samdb.attach_schema_from_ldif(head_data, schema_data) - + return schema_data; def get_schema_data(setup_path, subst_vars = None): """Get schema data from the AD schema files instead of schema.ldif. -- cgit