b94eda4466
5347b524-evtchn-eliminate-64k-ports-limitation.patch 53aac342-x86-HVM-consolidate-and-sanitize-CR4-guest-reserved-bit-determination.patch 53b16cd4-VT-d-ATS-correct-and-clean-up-dev_invalidate_iotlb.patch 53b56de1-properly-reference-count-DOMCTL_-un-pausedomain-hypercalls.patch 53cfdcc7-avoid-crash-when-doing-shutdown-with-active-cpupools.patch 53cfddaf-x86-mem_event-validate-the-response-vcpu_id-before-acting-on-it.patch 53cfdde4-x86-mem_event-prevent-underflow-of-vcpu-pause-counts.patch - bnc#886801 - xl vncviewer: The first domu can be accessed by any id 53c9151b-Fix-xl-vncviewer-accesses-port-0-by-any-invalid-domid.patch - Upstream pygrub bug fix 5370e03b-pygrub-fix-error-handling-if-no-valid-partitions-are-found.patch - Fix pygrub to handle old 32 bit VMs pygrub-boot-legacy-sles.patch (Mike Latimer) - Remove xen-vmresync utility. It is an old Platespin Orchestrate utility that should have never been included in the Xen package. Updated xen.spec - Rework xen-destroy utility included in xen-utils bnc#885292 and bnc#886063 Updated xen-utils-0.1.tar.bz2 - bnc#886063 - Xen monitor fails (xl list --long output different from xm list --long output) - bnc#885292 - VirtualDomain: pid_status does not know how to check status on SLE12 OBS-URL: https://build.opensuse.org/package/show/Virtualization/xen?expand=0&rev=322
40 lines
1.5 KiB
Diff
40 lines
1.5 KiB
Diff
Subject: xl: 'xl vncviewer' accesses port 0 by any invalid domid
|
|
From: Chunyan Liu cyliu@suse.com Fri Jul 18 14:18:04 2014 +0800
|
|
Date: Fri Jul 18 13:37:47 2014 +0100:
|
|
Git: 2a8cc1a55329ead252ed323ec4bbee534d5c0f23
|
|
|
|
Currently, with command:
|
|
xl vncviewer invalid_domid
|
|
it always brings user to the domU using vncport 5900.
|
|
The invalid domid could be an non-existing one or Dom0.
|
|
It's better to report error in this case.
|
|
|
|
Correct libxl_vncviewer_exec:
|
|
In existing code, when vncport is NULL, it still continues
|
|
and will show vncport 5900. So, with 'xl vncviewer 0' it also
|
|
wrongly shows domU using vncport 5900. Correct it to report error
|
|
if vncport is NULL.
|
|
|
|
Signed-off-by: Chunyan Liu <cyliu@suse.com>
|
|
Acked-by: Ian Campbell <ian.campbell@citrix.com>
|
|
|
|
Index: xen-4.4.0-testing/tools/libxl/libxl.c
|
|
===================================================================
|
|
--- xen-4.4.0-testing.orig/tools/libxl/libxl.c
|
|
+++ xen-4.4.0-testing/tools/libxl/libxl.c
|
|
@@ -1635,8 +1635,12 @@ int libxl_vncviewer_exec(libxl_ctx *ctx,
|
|
vnc_port = libxl__xs_read(gc, XBT_NULL,
|
|
libxl__sprintf(gc,
|
|
"/local/domain/%d/console/vnc-port", domid));
|
|
- if ( vnc_port )
|
|
- port = atoi(vnc_port) - 5900;
|
|
+ if (!vnc_port) {
|
|
+ LOG(ERROR, "Cannot get vnc-port of domain %d", domid);
|
|
+ goto x_fail;
|
|
+ }
|
|
+
|
|
+ port = atoi(vnc_port) - 5900;
|
|
|
|
vnc_listen = libxl__xs_read(gc, XBT_NULL,
|
|
libxl__sprintf(gc,
|