Change in vdsm[master]: virt: Fix of uninitialized variable access in Vm.setLinkAndN...

mzamazal at redhat.com mzamazal at redhat.com
Wed Feb 3 15:08:37 UTC 2016


Milan Zamazal has uploaded a new change for review.

Change subject: virt: Fix of uninitialized variable access in Vm.setLinkAndNetwork()
......................................................................

virt: Fix of uninitialized variable access in Vm.setLinkAndNetwork()

Change-Id: I04b44a1599593e825cc83f8df6edd4965ff43707
Signed-off-by: Milan Zamazal <mzamazal at redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/61/53061/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 4084d05..3d0724a 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -2146,7 +2146,7 @@
         try:
             link = vnicXML.getElementsByTagName('link')[0]
         except IndexError:
-            vnicXML.appendChildWithArgs('link')
+            link = vnicXML.appendChildWithArgs('link')
         link.setAttribute('state', linkValue)
         if (specParams and
                 ('inbound' in specParams or 'outbound' in specParams)):


-- 
To view, visit https://gerrit.ovirt.org/53061
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I04b44a1599593e825cc83f8df6edd4965ff43707
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Milan Zamazal <mzamazal at redhat.com>


More information about the vdsm-patches mailing list