46 lines
1.6 KiB
Diff
46 lines
1.6 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.
|
|
---
|
|
hw/xfree86/common/xf86Xinput.c | 23 +++++++++++++----------
|
|
1 files changed, 13 insertions(+), 10 deletions(-)
|
|
|
|
--- xorg-server-1.6.0/hw/xfree86/common/xf86Xinput.c.orig 2009-02-25 20:12:11.000000000 +0100
|
|
+++ xorg-server-1.6.0/hw/xfree86/common/xf86Xinput.c 2009-02-27 17:26:35.000000000 +0100
|
|
@@ -675,17 +675,20 @@ DeleteInputDeviceRequest(DeviceIntPtr pD
|
|
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)
|
|
{
|
|
- xfree(idev->driver);
|
|
- xfree(idev->identifier);
|
|
- xf86optionListFree(idev->commonOptions);
|
|
- xfree(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 */
|
|
+ {
|
|
+ xfree(idev->driver);
|
|
+ xfree(idev->identifier);
|
|
+ xf86optionListFree(idev->commonOptions);
|
|
+ xfree(idev);
|
|
+ }
|
|
}
|
|
}
|
|
OsReleaseSignals();
|