i8c
changed/i8c/xorg-x11-server-Xwayland-21.1.3-17.el8_10
parent
7d2e4eed55
commit
d3be865eeb
@ -0,0 +1,57 @@
|
|||||||
|
From 26120df7aae6b5bf8086fb4d871d3b1a07ddacdb Mon Sep 17 00:00:00 2001
|
||||||
|
From: Matthieu Herrb <matthieu@herrb.eu>
|
||||||
|
Date: Thu, 10 Oct 2024 10:37:28 +0200
|
||||||
|
Subject: [PATCH xserver] xkb: Fix buffer overflow in _XkbSetCompatMap()
|
||||||
|
MIME-Version: 1.0
|
||||||
|
Content-Type: text/plain; charset=UTF-8
|
||||||
|
Content-Transfer-Encoding: 8bit
|
||||||
|
|
||||||
|
The _XkbSetCompatMap() function attempts to resize the `sym_interpret`
|
||||||
|
buffer.
|
||||||
|
|
||||||
|
However, It didn't update its size properly. It updated `num_si` only,
|
||||||
|
without updating `size_si`.
|
||||||
|
|
||||||
|
This may lead to local privilege escalation if the server is run as root
|
||||||
|
or remote code execution (e.g. x11 over ssh).
|
||||||
|
|
||||||
|
CVE-2024-9632, ZDI-CAN-24756
|
||||||
|
|
||||||
|
This vulnerability was discovered by:
|
||||||
|
Jan-Niklas Sohn working with Trend Micro Zero Day Initiative
|
||||||
|
|
||||||
|
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
|
||||||
|
Tested-by: Peter Hutterer <peter.hutterer@who-t.net>
|
||||||
|
Reviewed-by: José Expósito <jexposit@redhat.com>
|
||||||
|
(cherry picked from commit 85b776571487f52e756f68a069c768757369bfe3)
|
||||||
|
|
||||||
|
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1735>
|
||||||
|
---
|
||||||
|
xkb/xkb.c | 8 ++++----
|
||||||
|
1 file changed, 4 insertions(+), 4 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/xkb/xkb.c b/xkb/xkb.c
|
||||||
|
index 8d52e25df..8b63e34b5 100644
|
||||||
|
--- a/xkb/xkb.c
|
||||||
|
+++ b/xkb/xkb.c
|
||||||
|
@@ -2990,13 +2990,13 @@ _XkbSetCompatMap(ClientPtr client, DeviceIntPtr dev,
|
||||||
|
XkbSymInterpretPtr sym;
|
||||||
|
unsigned int skipped = 0;
|
||||||
|
|
||||||
|
- if ((unsigned) (req->firstSI + req->nSI) > compat->num_si) {
|
||||||
|
- compat->num_si = req->firstSI + req->nSI;
|
||||||
|
+ if ((unsigned) (req->firstSI + req->nSI) > compat->size_si) {
|
||||||
|
+ compat->num_si = compat->size_si = req->firstSI + req->nSI;
|
||||||
|
compat->sym_interpret = reallocarray(compat->sym_interpret,
|
||||||
|
- compat->num_si,
|
||||||
|
+ compat->size_si,
|
||||||
|
sizeof(XkbSymInterpretRec));
|
||||||
|
if (!compat->sym_interpret) {
|
||||||
|
- compat->num_si = 0;
|
||||||
|
+ compat->num_si = compat->size_si = 0;
|
||||||
|
return BadAlloc;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
--
|
||||||
|
2.47.0
|
||||||
|
|
Loading…
Reference in new issue