diff options
| author | Martin Basti <mbasti@redhat.com> | 2016-06-20 12:48:38 +0200 |
|---|---|---|
| committer | Martin Basti <mbasti@redhat.com> | 2016-06-20 20:49:35 +0200 |
| commit | fe689e9938d6bcddd848f70b2f719253c55c09d7 (patch) | |
| tree | 38c962ae1ca47bdd7e52c93fbbb326e4b5b300c4 /ipaserver/install | |
| parent | b6bab8d4e0d6f4715ef353b6944c85c5e88d44ab (diff) | |
| download | freeipa-fe689e9938d6bcddd848f70b2f719253c55c09d7.tar.gz freeipa-fe689e9938d6bcddd848f70b2f719253c55c09d7.tar.xz freeipa-fe689e9938d6bcddd848f70b2f719253c55c09d7.zip | |
Fix possibly undefined variable in ipa_smb_conf_exists()
There was missing else statement what may result in undefined conf_fd
variable.
Reviewed-By: Alexander Bokovoy <abokovoy@redhat.com>
Diffstat (limited to 'ipaserver/install')
| -rw-r--r-- | ipaserver/install/adtrustinstance.py | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/ipaserver/install/adtrustinstance.py b/ipaserver/install/adtrustinstance.py index 944741221..6ab15df27 100644 --- a/ipaserver/install/adtrustinstance.py +++ b/ipaserver/install/adtrustinstance.py @@ -81,6 +81,8 @@ def ipa_smb_conf_exists(): except IOError as err: if err.errno == errno.ENOENT: return False + else: + raise lines = conf_fd.readlines() conf_fd.close() |