Change in vdsm[master]: virt: sampling: make sure balloon values are str

fromani at redhat.com fromani at redhat.com
Thu Jun 19 14:50:39 UTC 2014


Francesco Romani has uploaded a new change for review.

Change subject: virt: sampling: make sure balloon values are str
......................................................................

virt: sampling: make sure balloon values are str

This patch avoids XML-RPC integer overflows by ensuring
all the balloon stats are string values.

Change-Id: Ia71ceab52c1247a94ab3a61b3bbafec76106d789
Signed-off-by: Francesco Romani <fromani 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/54/28954/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index ba91635..d61d171 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -373,7 +373,7 @@
             'balloon_min': str(
                 int(self._vm.conf.get('memGuaranteedSize', '0')) * 1024),
             'balloon_cur': str(sInfo) if sInfo is not None else '0',
-            'balloon_target': balloon_target}
+            'balloon_target': str(balloon_target)}
 
     def _getCpuTuneInfo(self, stats):
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia71ceab52c1247a94ab3a61b3bbafec76106d789
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>


More information about the vdsm-patches mailing list