diff options
author | Christian Heimes <cheimes@redhat.com> | 2018-01-03 10:21:46 +0100 |
---|---|---|
committer | Christian Heimes <cheimes@redhat.com> | 2018-01-09 07:53:28 +0100 |
commit | de616888d5718532dbb8e66ed3f6c698cdf1e79c (patch) | |
tree | de678921c11ac146cc8fad0d5acc2f7f0937cf24 | |
parent | d3f43a673771aa40365ac1b34dc21e4e22d4a7cb (diff) | |
download | freeipa-de616888d5718532dbb8e66ed3f6c698cdf1e79c.tar.gz freeipa-de616888d5718532dbb8e66ed3f6c698cdf1e79c.tar.xz freeipa-de616888d5718532dbb8e66ed3f6c698cdf1e79c.zip |
LGTM: Fix exception in permission_del
Instantiating an exception, but not raising it, has no effect.
https://pagure.io/freeipa/issue/7344
Signed-off-by: Christian Heimes <cheimes@redhat.com>
Reviewed-By: Fraser Tweedale <ftweedal@redhat.com>
-rw-r--r-- | ipaserver/plugins/permission.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ipaserver/plugins/permission.py b/ipaserver/plugins/permission.py index a91426afd..49ac7cfca 100644 --- a/ipaserver/plugins/permission.py +++ b/ipaserver/plugins/permission.py @@ -1072,7 +1072,7 @@ class permission_del(baseldap.LDAPDelete): try: self.obj.remove_aci(entry) except errors.NotFound: - errors.NotFound( + raise errors.NotFound( reason=_('ACI of permission %s was not found') % keys[0]) return dn |