Bug 69490 - SIGSEGV - Text input protocol attempts to post event to dead or dying resource
Summary: SIGSEGV - Text input protocol attempts to post event to dead or dying resource
Status: VERIFIED FIXED
Alias: None
Product: Wayland
Classification: Unclassified
Component: weston (show other bugs)
Version: 1.2.x
Hardware: Other All
: medium critical
Assignee: Wayland bug list
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-09-17 20:28 UTC by U. Artie Eoff
Modified: 2014-01-13 23:10 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
Resource is null (12.83 KB, text/plain)
2013-09-17 20:28 UTC, U. Artie Eoff
Details
Resource interface events access violation (13.42 KB, text/plain)
2013-09-17 20:29 UTC, U. Artie Eoff
Details
resource interface event signature is null (13.36 KB, text/plain)
2013-09-17 20:30 UTC, U. Artie Eoff
Details

Description U. Artie Eoff 2013-09-17 20:28:05 UTC
There seems to be a race between a client disconnecting and the text protocol attempting to post an event to the dying client resource via wl_text_input_send_language(...).  Various backtraces indicate that the resource is not always completely destructed before it reaches wl_resource_post_event(...), but ultimately results in a segmentation fault.  See attached backtraces for more evidence of this issue.

wayland (master) heads/master-0-g9a5ed78
drm (master) heads/master-0-gb6da447
mesa (master) heads/master-0-ga1b6e69
libva (master) heads/master-0-g73a11b3
intel-driver (master) heads/master-0-gd22d367
weston (master) heads/master-0-g4a4704a

The only existing test case, so far, that reproduces this issue is the efl/shm/EntryCutTest in wayland-fits.  Nonetheless, this appears to be a core issue unrelated to efl.
Comment 1 U. Artie Eoff 2013-09-17 20:28:46 UTC
Created attachment 86017 [details]
Resource is null
Comment 2 U. Artie Eoff 2013-09-17 20:29:34 UTC
Created attachment 86018 [details]
Resource interface events access violation
Comment 3 U. Artie Eoff 2013-09-17 20:30:06 UTC
Created attachment 86019 [details]
resource interface event signature is null
Comment 4 U. Artie Eoff 2013-11-13 21:51:10 UTC
I can't seem to reproduce this one anymore with wayland/weston > 1.3 and latest master components:

wayland (master) heads/master-0-g05f95c8
drm (master) heads/master-0-g1a84eea
mesa (master) heads/master-0-g19c2f40
libva (master) heads/master-0-g73a11b3
intel-driver (master) heads/master-0-g46c4901
weston (master) heads/master-0-gf707e81
efl (master) heads/master-0-g90bbc21
elementary (master) heads/master-0-g4832558
wfits (master) heads/master-0-gbb9959e

...and I was unsuccessful in identifying which change could have fixed it :-/
Comment 5 U. Artie Eoff 2013-12-02 22:30:07 UTC
Ok, I'm seeing this one again...

wayland (master) heads/master-0-g5a019e3
drm (master) heads/master-0-gc3d9689
mesa (master) heads/master-0-gdecf070
libva (master) heads/master-0-g73a11b3
intel-driver (master) heads/master-0-g4a0f76c
weston (master) heads/master-0-gda704d9
wfits (master) heads/master-0-g887e47f
Comment 6 Kristian Høgsberg 2014-01-13 23:06:46 UTC
commit c180977e7cfd36eb8286186ca9432b1d4cfef864
Author: Kristian Høgsberg <krh@bitplanet.net>
Date:   Mon Jan 13 15:06:10 2014 -0800

    text: Set context->model to NULL when we deactivate text input
    
    There's a small window between the input method (eg the on-sreen keyboard)
    receiving the deactivate and destroying the context, where the keyboard may
    send requests, which we forward to the destroyed input method.  Fix this
    by setting the contexts model to NULL right away and then avoid sending
    events if context->model is NULL.
    
    Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=69490


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.