diff options
author | Martin Basti <mbasti@redhat.com> | 2016-05-18 18:51:04 +0200 |
---|---|---|
committer | Martin Basti <mbasti@redhat.com> | 2016-05-20 15:05:25 +0200 |
commit | ad1cac12834615a4666624885ae4997286641548 (patch) | |
tree | 61e7aac9f9c21f9aeab5a5c54749510668d70022 /ipalib | |
parent | 6e4b749b59ebae82c613fe799dda7cb21dc080cd (diff) | |
download | freeipa-ad1cac12834615a4666624885ae4997286641548.tar.gz freeipa-ad1cac12834615a4666624885ae4997286641548.tar.xz freeipa-ad1cac12834615a4666624885ae4997286641548.zip |
Fix referenced before assigment variables in except statements
Variable msg may not exists in the last except context, and even it
contains improper value because it is not related to catched exception
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
Diffstat (limited to 'ipalib')
-rw-r--r-- | ipalib/plugins/baseldap.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ipalib/plugins/baseldap.py b/ipalib/plugins/baseldap.py index dc61fe454..4f90291cb 100644 --- a/ipalib/plugins/baseldap.py +++ b/ipalib/plugins/baseldap.py @@ -2215,7 +2215,7 @@ class LDAPAddReverseMember(LDAPModReverseMember): failed['member'][self.reverse_attr].append((attr, unicode(msg.strip()))) except errors.PublicError as e: - failed['member'][self.reverse_attr].append((attr, unicode(msg))) + failed['member'][self.reverse_attr].append((attr, unicode(e))) # Update the member data. entry_attrs = ldap.get_entry(dn, ['*']) @@ -2316,7 +2316,7 @@ class LDAPRemoveReverseMember(LDAPModReverseMember): failed['member'][self.reverse_attr].append((attr, unicode(msg.strip()))) except errors.PublicError as e: - failed['member'][self.reverse_attr].append((attr, unicode(msg))) + failed['member'][self.reverse_attr].append((attr, unicode(e))) # Update the member data. entry_attrs = ldap.get_entry(dn, ['*']) |