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