[Xen-changelog] [xen-unstable] xend: xenapi: Suspended domain causes fault if vif.get_all_records() is called

[Xen patchbot-unstable <patchbot-unstable@xxxxxxxxxxxxxxxxxxx>]

# HG changeset patch
# User Keir Fraser <keir@xxxxxxxxxxxxx>
# Date 1192552893 -3600
# Node ID a38a2fdb4f1ba55907bc508054e3205f413a3dc2
# Parent  b6ff4859b3d4a78edd068b2c6c86aa8964694b8d
xend: xenapi: Suspended domain causes fault if vif.get_all_records() is called

A single suspended domain on the system causes a fault when
vif.get_all_records() is called since this returns an ErrorDescription
and no 'Value' in the 'v' dictionary. This patch now returns a 'None'
as Value which might not be optimal but better than faulting.

Signed-off-by: Stefan Berger <stefanb@xxxxxxxxxx>
---
 tools/python/xen/xend/XendAPI.py |    2 +-
 1 files changed, 1 insertion(+), 1 deletion(-)

diff -r b6ff4859b3d4 -r a38a2fdb4f1b tools/python/xen/xend/XendAPI.py
--- a/tools/python/xen/xend/XendAPI.py  Tue Oct 16 17:40:22 2007 +0100
+++ b/tools/python/xen/xend/XendAPI.py  Tue Oct 16 17:41:33 2007 +0100
@@ -555,7 +555,7 @@ class XendAPI(object):
                 return xen_api_success(ref)
 
             def unpack(v):
-                return v['Value']
+                return v.get('Value')
 
             def _get_all_records(_api_cls):
                 return lambda s, session: \

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog