diff options
author | Amitay Isaacs <amitay@gmail.com> | 2011-08-22 09:43:20 +1000 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2011-08-26 10:06:32 +1000 |
commit | 7f67d7b5caa0915820b9ae3f9b83b7b02c63b307 (patch) | |
tree | efb2db11d5a17a6ebfee254d73f829406307d71f /source4/scripting/python/samba | |
parent | 886203f3bc2ed445c6b3a03dbe145ecaa1b44050 (diff) | |
download | samba-7f67d7b5caa0915820b9ae3f9b83b7b02c63b307.tar.gz samba-7f67d7b5caa0915820b9ae3f9b83b7b02c63b307.tar.bz2 samba-7f67d7b5caa0915820b9ae3f9b83b7b02c63b307.zip |
s3_upgrade: Let python generate backtrace for unknown exceptions
Catch known exceptions only.
Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Diffstat (limited to 'source4/scripting/python/samba')
-rw-r--r-- | source4/scripting/python/samba/upgrade.py | 19 |
1 files changed, 4 insertions, 15 deletions
diff --git a/source4/scripting/python/samba/upgrade.py b/source4/scripting/python/samba/upgrade.py index b71b1d7d7c..62bda48aca 100644 --- a/source4/scripting/python/samba/upgrade.py +++ b/source4/scripting/python/samba/upgrade.py @@ -162,16 +162,11 @@ def add_idmap_entry(idmapdb, sid, xid, xid_type, logger): # First try to see if we already have this entry found = False - try: - msg = idmapdb.search(expression='objectSid=%s' % str(sid)) - if msg.count == 1: - found = True - except Exception, e: - raise e + msg = idmapdb.search(expression='objectSid=%s' % str(sid)) + if msg.count == 1: + found = True if found: - print msg.count - print dir(msg) try: m = ldb.Message() m.dn = ldb.Dn(idmapdb, msg[0]['dn']) @@ -181,8 +176,6 @@ def add_idmap_entry(idmapdb, sid, xid, xid_type, logger): except ldb.LdbError, e: logger.warn('Could not modify idmap entry for sid=%s, id=%s, type=%s (%s)', str(sid), str(xid), xid_type, str(e)) - except Exception, e: - raise e else: try: idmapdb.add({"dn": "CN=%s" % str(sid), @@ -243,8 +236,6 @@ def add_group_from_mapping_entry(samdb, groupmap, logger): found = False else: raise ldb.LdbError(ecode, emsg) - except Exception, e: - raise e if found: logger.warn('Group already exists sid=%s, groupname=%s existing_groupname=%s, Ignoring.', @@ -287,8 +278,6 @@ def add_users_to_group(samdb, group, members, logger): samdb.modify(m) except ldb.LdbError, e: logger.warn("Could not add member to group '%s'", groupmap.nt_name) - except Exception, e: - raise(e) def import_wins(samba4_winsdb, samba3_winsdb): @@ -554,7 +543,7 @@ def upgrade_from_samba3(samba3, logger, session_info, smbconf, targetdir): # Get domain sid try: domainsid = passdb.get_global_sam_sid() - except: + except passdb.error: raise Exception("Can't find domain sid for '%s', Exiting." % domainname) # Get machine account, sid, rid |