]> xenbits.xensource.com Git - xen.git/commitdiff
tools/xc_restore: Initialise console and store mfns
authorAndrew Cooper <andrew.cooper3@citrix.com>
Mon, 25 Nov 2013 11:05:47 +0000 (11:05 +0000)
committerIan Jackson <Ian.Jackson@eu.citrix.com>
Mon, 25 Nov 2013 12:05:34 +0000 (12:05 +0000)
If the console or store mfn chunks are not present in the migration stream,
stack junk gets reported for the mfns.

XenServer had a very hard to track down VM corruption issue caused by exactly
this issue.  Xenconsoled would connect to a junk mfn and incremented the ring
pointer if the junk happend to look like a valid gfn.

Coverity ID: 1056093 1056094

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
CC: Ian Campbell <Ian.Campbell@citrix.com>
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
tools/xcutils/xc_restore.c

index 5ec90ac8adee8b00b19827b84dbd87b1bfa4a9a7..4ea261b7088c7be8d39d942767a1eb1c1c3ac9ce 100644 (file)
@@ -23,7 +23,7 @@ main(int argc, char **argv)
     xc_interface *xch;
     int io_fd, ret;
     int superpages;
-    unsigned long store_mfn, console_mfn;
+    unsigned long store_mfn = 0, console_mfn = 0;
     xentoollog_level lvl;
     xentoollog_logger *l;