forked from pool/xorg-x11-server
Stefan Dirsch
c8690bb170
This submit is just for preparing the final 13.0 release, please have a look at it and comment so i can fix things up until the final 13.0 release for 12.3! #3 OBS-URL: https://build.opensuse.org/request/show/131324 OBS-URL: https://build.opensuse.org/package/show/X11:XOrg/xorg-x11-server?expand=0&rev=408
43 lines
1.4 KiB
Diff
43 lines
1.4 KiB
Diff
>From 829037395f8b93e69a30852a95e378f78c3ccd6b Mon Sep 17 00:00:00 2001
|
|
From: Luc Verhaegen <libv@skynet.be>
|
|
Date: Wed, 12 Nov 2008 17:09:33 +0100
|
|
Subject: [PATCH] Xinput: Catch missing configlayout when deleting device.
|
|
|
|
In DeleteInputDeviceRequest (xf86Xinput.c), we access idev members
|
|
even if idev is null. This takes down the xserver hard in some cases
|
|
(kernel SIGABRT), and segfaults on other cases.
|
|
================================================================================
|
|
--- xorg-server-1.7.99/hw/xfree86/common/xf86Xinput.c
|
|
+++ xorg-server-1.7.99/hw/xfree86/common/xf86Xinput.c
|
|
@@ -870,17 +870,20 @@
|
|
else
|
|
xf86DeleteInput(pInfo, 0);
|
|
|
|
- /* devices added through HAL aren't in the config layout */
|
|
- it = xf86ConfigLayout.inputs;
|
|
- while(*it && *it != idev)
|
|
- it++;
|
|
-
|
|
- if (!(*it)) /* end of list, not in the layout */
|
|
+ if (idev)
|
|
{
|
|
- free(idev->driver);
|
|
- free(idev->identifier);
|
|
- xf86optionListFree(idev->commonOptions);
|
|
- free(idev);
|
|
+ /* devices added through HAL aren't in the config layout */
|
|
+ it = xf86ConfigLayout.inputs;
|
|
+ while(*it && *it != idev)
|
|
+ it++;
|
|
+
|
|
+ if (!(*it)) /* end of list, not in the layout */
|
|
+ {
|
|
+ free(idev->driver);
|
|
+ free(idev->identifier);
|
|
+ xf86optionListFree(idev->commonOptions);
|
|
+ free(idev);
|
|
+ }
|
|
}
|
|
}
|
|
OsReleaseSignals();
|