]> xenbits.xensource.com Git - qemu-xen-4.1-testing.git/commitdiff
Disable qemu monitor by default. The qemu monitor is an overly
authorIan Jackson <ian.jackson@eu.citrix.com>
Thu, 6 Sep 2012 16:05:30 +0000 (17:05 +0100)
committerIan Jackson <Ian.Jackson@eu.citrix.com>
Thu, 6 Sep 2012 16:07:47 +0000 (17:07 +0100)
powerful feature which must be protected from untrusted (guest)
administrators.

Neither xl nor xend expect qemu to produce this monitor unless it is
explicitly requested.

This is a security problem, XSA-19.  Previously it was CVE-2007-0998
in Red Hat but we haven't dealt with it in upstream.  We hope to have
a new CVE for it here but we don't have one yet.

Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
(cherry picked from commit bacc0d302445c75f18f4c826750fb5853b60e7ca)

vl.c

diff --git a/vl.c b/vl.c
index f07a6592ac84954a4ce59b1dd04d96d2c2642b64..686a9bdd586c9c3843e430275e034f9a62460398 100644 (file)
--- a/vl.c
+++ b/vl.c
@@ -4910,7 +4910,7 @@ int main(int argc, char **argv, char **envp)
     kernel_cmdline = "";
     cyls = heads = secs = 0;
     translation = BIOS_ATA_TRANSLATION_AUTO;
-    monitor_device = "vc:80Cx24C";
+    monitor_device = "null";
 
     serial_devices[0] = "vc:80Cx24C";
     for(i = 1; i < MAX_SERIAL_PORTS; i++)