]> xenbits.xensource.com Git - xen.git/commitdiff
libxl: fix libxl_set_memory_target
authorWei Liu <wei.liu2@citrix.com>
Thu, 29 Dec 2016 16:36:31 +0000 (16:36 +0000)
committerIan Jackson <Ian.Jackson@eu.citrix.com>
Fri, 20 Jan 2017 17:38:45 +0000 (17:38 +0000)
Commit 26dbc93a ("libxl: Remove pointless hypercall from
libxl_set_memory_target") removed the call to xc_domain_getinfolist, but
it failed to notice that "info" was actually needed later.

Put that back. While at it, make the code conform to coding style
requirement.

Reported-by: Juergen Gross <jgross@suse.com>
Signed-off-by: Wei Liu <wei.liu2@citrix.com>
(cherry picked from commit ed5f19aea66fe5a72060d6a795ffcd23b7643ee3)
(cherry picked from commit e1cefedd80f9972854769bfc6e32e23b56cd0712)
(cherry picked from commit 013ee593ca0456e1adfdf80ef7e44b151cdb1545)
(cherry picked from commit 3658f7a0bdd8fcda217559927e25263a07398c27)

tools/libxl/libxl.c

index aa39e78c249c41817af150b976d56e7bdf9125b5..910fa6847ce05e28926b627c6c578acd7ba8cb47 100644 (file)
@@ -4937,6 +4937,13 @@ retry_transaction:
         goto out;
     }
 
+    r = xc_domain_getinfolist(ctx->xch, domid, 1, &info);
+    if (r != 1 || info.domain != domid) {
+        abort_transaction = 1;
+        rc = ERROR_FAIL;
+        goto out;
+    }
+
     libxl_dominfo_init(&ptr);
     xcinfo2xlinfo(ctx, &info, &ptr);
     uuid = libxl__uuid2string(gc, ptr.uuid);