Bug 49598 - Crash at exit due to recent ReleaseButtonsAndKeys addition to 1.12 branch
Summary: Crash at exit due to recent ReleaseButtonsAndKeys addition to 1.12 branch
Alias: None
Product: xorg
Classification: Unclassified
Component: Server/General (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Peter Hutterer
QA Contact: Xorg Project Team
URL: http://bugs.debian.org/671812
Keywords: have-backtrace, patch, regression
Depends on:
Reported: 2012-05-07 11:32 UTC by Cyril Brulebois
Modified: 2012-05-20 01:23 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:

gdb backtrace from Jakub Wilk (32.92 KB, text/plain)
2012-05-07 11:32 UTC, Cyril Brulebois
no flags Details

Description Cyril Brulebois 2012-05-07 11:32:40 UTC
Created attachment 61167 [details]
gdb backtrace from Jakub Wilk


crash with X 1.12 at exit time, with current server-1.12-branch, due to:

commit 90299556db24543bb7365e8c2897deca3aa219e7
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Mon Apr 30 10:01:48 2012 +1000

    dix: when disabling a device, release all buttons and keys
    A suspend-induced device disable may happen before the device gets to see
    the button release event. On resume, the server's internal state still has
    some buttons pressed, causing inconsistent behaviour.
    Force the release and the matching events to be sent to the client.
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Chase Douglas <chase.douglas@canonical.com>
    (cherry picked from commit f3410b97cf9b48a47bee3d15d232f8a88e75f4ef)

Indeed, as Julien mentioned on IRC, the codepath looks like it, and a revert on top of 22a1953c4a2747d637bb926034f11134809b8d06 makes my 1.12 branch stop crashing.

Full backtrace attached, but also available in:
Comment 1 Julien Cristau 2012-05-09 13:24:02 UTC
Comment 2 Cyril Brulebois 2012-05-20 01:23:21 UTC
Reverted as of: xorg-server-

commit 58dfb13953af71021317b9d85230b1163198f031
Author: Jeremy Huddleston <jeremyhu@apple.com>
Date:   Thu May 10 00:53:20 2012 -0700

    Revert "dix: when disabling a device, release all buttons and keys"
    This reverts commit 90299556db24543bb7365e8c2897deca3aa219e7.
    The commit being reverted triggered a segfault on server shutdown when a
    device posts raw events after the root windows are forced to NULL.
    Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

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.