[node-patches] Change in ovirt-node[master]: ovirtfunctions: Add validation to /liveos/version

dougsland at redhat.com dougsland at redhat.com
Tue Aug 26 14:53:52 UTC 2014


Douglas Schilling Landgraf has uploaded a new change for review.

Change subject: ovirtfunctions: Add validation to /liveos/version
......................................................................

ovirtfunctions: Add validation to /liveos/version

Before accessing /liveos/version we should make sure
it exists.

Change-Id: I3192214d29f267263c24523335c4ffd3501f4128
Signed-off-by: Douglas Schilling Landgraf <dougsland at redhat.com>
---
M src/ovirtnode/ovirtfunctions.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/98/31998/1

diff --git a/src/ovirtnode/ovirtfunctions.py b/src/ovirtnode/ovirtfunctions.py
index 1d53297..5fd352d 100644
--- a/src/ovirtnode/ovirtfunctions.py
+++ b/src/ovirtnode/ovirtfunctions.py
@@ -1486,7 +1486,7 @@
         system_closefds(cmd)
 
 def get_installed_version_number():
-    if mount_liveos():
+    if mount_liveos() and os.path.exists("/liveos/version"):
         existing_version = open("/liveos/version")
         existing_install = {}
         for line in existing_version.readlines():


-- 
To view, visit http://gerrit.ovirt.org/31998
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3192214d29f267263c24523335c4ffd3501f4128
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>



More information about the node-patches mailing list