AbortDDX() fails to check whether xf86Info.kbdProc is non-NULL before attempting to call it. This can result in a double segfault, leaving the console locked.
Created attachment 1616 [details] [review] Patch
This was fixed in HEAD already, just needs to be backported to the 6.8.x branch. The fix in head was commited as: 2004-09-21 Egbert Eich <eich-at-freedesktop-org> * programs/Xserver/hw/xfree86/common/xf86Init.c: (AbortDDX): Check if xf86Info.kbdProc pointer is really set before calling it on abort as this pointer won't be set if the new modular keyboard driver is used (Matthias Hopf).
Alan Coopersmith wrote: > This was fixed in HEAD already, just needs to be backported to the 6.8.x > branch. The fix in head was commited as: Alan: Correct me if I am wrong - the patch has already been applied to the X11R6.8.x stable branch... right ?
Comment on attachment 1616 [details] [review] Patch Whoops - you're right - this was already included in Bug #1933/attachment #1392 [details] [review]. Guess I was looking at an out of date copy of the tree when I looked at this before. Sorry.
*** This bug has been marked as a duplicate of 1933 ***
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.