From 09cb7b2d2aa35c6353202e57867f1970df9f5ec2 Mon Sep 17 00:00:00 2001 From: Julien Cristau Date: Fri, 3 Jul 2015 23:53:25 +0200 Subject: nx-X11: Backport: randr: fix server crash in RRGetScreenInfo We don't return rates to randr < 1.1 clients, so don't allocate space for them. This fixes a FatalError due to not all allocated space being used. X.Org bug#21861 Reported-by: Guillaume Quintin Signed-off-by: Julien Cristau Backported from Arctica GH 3.6.x branch. v2: backport to nx-libs 3.6.x (Ulrich Sibiller) v3: backport to nx-libs 3.5.0.x (Mihai Moldovan) --- debian/changelog | 6 ++++ ...r-fix-server-crash-in-RRGetScreenInf.full.patch | 36 ++++++++++++++++++++++ debian/patches/series | 1 + 3 files changed, 43 insertions(+) create mode 100644 debian/patches/1254_nx-X11_randr-fix-server-crash-in-RRGetScreenInf.full.patch diff --git a/debian/changelog b/debian/changelog index 3802f15c3..7b82e2d3a 100644 --- a/debian/changelog +++ b/debian/changelog @@ -287,6 +287,12 @@ nx-libs (2:3.5.0.32-0x2go1) UNRELEASED; urgency=low v3: backport to nx-libs 3.5.0.x (Mihai Moldovan) Adds: - 1253_nx-X11_randr-check-for-virtual-size-limits-befo.full.patch + * nx-X11: randr: fix server crash in RRGetScreenInfo + Backported from Arctica GH 3.6.x branch. + v2: backport to nx-libs 3.6.x (Ulrich Sibiller) + v3: backport to nx-libs 3.5.0.x (Mihai Moldovan) + Adds: + - 1254_nx-X11_randr-fix-server-crash-in-RRGetScreenInf.full.patch -- X2Go Release Manager Tue, 17 Mar 2015 19:19:32 +0100 diff --git a/debian/patches/1254_nx-X11_randr-fix-server-crash-in-RRGetScreenInf.full.patch b/debian/patches/1254_nx-X11_randr-fix-server-crash-in-RRGetScreenInf.full.patch new file mode 100644 index 000000000..cab084ddd --- /dev/null +++ b/debian/patches/1254_nx-X11_randr-fix-server-crash-in-RRGetScreenInf.full.patch @@ -0,0 +1,36 @@ +commit faddfbec6ff331c3859e2f57040da4a8cf072baa +Author: Julien Cristau +Date: Fri May 22 09:54:38 2009 +0200 + + Backport: randr: fix server crash in RRGetScreenInfo + + We don't return rates to randr < 1.1 clients, so don't allocate space + for them. This fixes a FatalError due to not all allocated space being + used. + + X.Org bug#21861 + + Reported-by: Guillaume Quintin + Signed-off-by: Julien Cristau + + Backported from Arctica GH 3.6.x branch. + + v2: backport to nx-libs 3.6.x (Ulrich Sibiller) + v3: backport to nx-libs 3.5.0.x (Mihai Moldovan) + +diff --git a/nx-X11/programs/Xserver/randr/rrscreen.c b/nx-X11/programs/Xserver/randr/rrscreen.c +index 9bea97b..20eed11 100644 +--- a/nx-X11/programs/Xserver/randr/rrscreen.c ++++ b/nx-X11/programs/Xserver/randr/rrscreen.c +@@ -677,8 +677,9 @@ ProcRRGetScreenInfo (ClientPtr client) + rep.sizeID = pData->size; + rep.rate = pData->refresh; + +- extraLen = (rep.nSizes * sizeof (xScreenSizes) + +- rep.nrateEnts * sizeof (CARD16)); ++ extraLen = rep.nSizes * sizeof (xScreenSizes); ++ if (has_rate) ++ extraLen += rep.nrateEnts * sizeof (CARD16); + + if (extraLen) + { diff --git a/debian/patches/series b/debian/patches/series index 61a638224..8169c803a 100644 --- a/debian/patches/series +++ b/debian/patches/series @@ -140,6 +140,7 @@ 1251_nx-X11_Drop-a-reference-to-user-mode-after-crea.full.patch 1252_nx-X11_Free-randr-crtc-and-output-pointer-array.full.patch 1253_nx-X11_randr-check-for-virtual-size-limits-befo.full.patch +1254_nx-X11_randr-fix-server-crash-in-RRGetScreenInf.full.patch 9900-dxpc-license-history.full+lite.patch 0016_nx-X11_install-location.debian.patch 0102_xserver-xext_set-securitypolicy-path.debian.patch -- cgit v1.2.3