Francesco Romani has uploaded a new change for review.

Change subject: vm: conf: proper locking in onConnect()
......................................................................

vm: conf: proper locking in onConnect()

Change-Id: I5f75630e204708bb4352f68240550b2b454806cc
Bug-Url: https://bugzilla.redhat.com/1318550
Signed-off-by: Francesco Romani <from...@redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/07/54907/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 956c8c3..955b8ad 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -845,7 +845,8 @@
 
     def onConnect(self, clientIp='', clientPort=''):
         if clientIp:
-            self.conf['clientIp'] = clientIp
+            with self._confLock:
+                self.conf['clientIp'] = clientIp
             self._clientPort = clientPort
 
     def _timedDesktopLock(self):


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5f75630e204708bb4352f68240550b2b454806cc
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to