diff options
| author | Jelmer Vernooij <jelmer@samba.org> | 2012-02-26 20:42:41 +0100 |
|---|---|---|
| committer | Jelmer Vernooij <jelmer@samba.org> | 2012-02-26 20:52:05 +0100 |
| commit | 6b320d63f3cc13de979739963e19a8d2a28ce135 (patch) | |
| tree | fbe504095848864dd864ea0e56756c0cdbc54730 /source4 | |
| parent | 171dc5ad4f10bca0038653f770decc72b872220d (diff) | |
| download | samba-6b320d63f3cc13de979739963e19a8d2a28ce135.tar.gz samba-6b320d63f3cc13de979739963e19a8d2a28ce135.tar.bz2 samba-6b320d63f3cc13de979739963e19a8d2a28ce135.zip | |
provision: Properly close opened files.
Diffstat (limited to 'source4')
| -rw-r--r-- | source4/scripting/python/samba/provision/__init__.py | 20 |
1 files changed, 14 insertions, 6 deletions
diff --git a/source4/scripting/python/samba/provision/__init__.py b/source4/scripting/python/samba/provision/__init__.py index e627c86a50..bff3ce6bf1 100644 --- a/source4/scripting/python/samba/provision/__init__.py +++ b/source4/scripting/python/samba/provision/__init__.py @@ -680,8 +680,10 @@ def make_smbconf(smbconf, hostname, domain, realm, serverrole, # this ensures that any smb.conf parameters that were set # on the provision/join command line are set in the resulting smb.conf f = open(smbconf, mode='w') - lp.dump(f, False) - f.close() + try: + lp.dump(f, False) + finally: + f.close() def setup_name_mappings(idmap, sid, root_uid, nobody_uid, @@ -1069,8 +1071,11 @@ def getpolicypath(sysvolpath, dnsdomain, guid): def create_gpo_struct(policy_path): if not os.path.exists(policy_path): os.makedirs(policy_path, 0775) - open(os.path.join(policy_path, "GPT.INI"), 'w').write( - "[General]\r\nVersion=0") + f = open(os.path.join(policy_path, "GPT.INI"), 'w') + try: + f.write("[General]\r\nVersion=0") + finally: + f.close() p = os.path.join(policy_path, "MACHINE") if not os.path.exists(p): os.makedirs(p, 0775) @@ -1651,8 +1656,11 @@ def provision(logger, session_info, credentials, smbconf=None, # if Samba Team members can't figure out the weird errors # loading an empty smb.conf gives, then we need to be smarter. # Pretend it just didn't exist --abartlet - data = open(smbconf, 'r').read() - data = data.lstrip() + f = open(smbconf, 'r') + try: + data = f.read().lstrip() + finally: + f.close() if data is None or data == "": make_smbconf(smbconf, hostname, domain, realm, serverrole, targetdir, sid_generator, useeadb, |