On Mon, 20 Jun 2016, Martin Basti wrote:
Patch attached.


From a073b44587a5b34c4f1de5742d54e7c547cd5821 Mon Sep 17 00:00:00 2001
From: Martin Basti <mba...@redhat.com>
Date: Mon, 20 Jun 2016 12:48:38 +0200
Subject: [PATCH] Fix possibly undefined variable in ipa_smb_conf_exists()

There was missing else statement what may result in undefined conf_fd
variable.
---
ipaserver/install/adtrustinstance.py | 2 ++
1 file changed, 2 insertions(+)

diff --git a/ipaserver/install/adtrustinstance.py 
b/ipaserver/install/adtrustinstance.py
index 
94474122125a59d7da8b05a13dcd6c0f20568855..6ab15df27216580d440ce72386113d6872c046b2
 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()
--
2.5.5

ACK, thanks!

--
/ Alexander Bokovoy

--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to