[Wayland-bugs] [Bug 72839] SIGSEGV in Weston run_touch_binding

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=72839

Kristian Høgsberg <krh at bitplanet.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #5 from Kristian Høgsberg <krh at bitplanet.net> ---
Those touch coordinates in the backtrace look really out of place, that may be
a different problem in itself.  But there are a few corner cases where we can
legitimately fail to touch a surface, so we need to check for NULL in
touch_to_activate_binding().

commit 0ed6750541eac94af7083fabe5911c20e6ca31f9
Author: Kristian Høgsberg <krh at bitplanet.net>
Date:   Thu Jan 2 23:00:11 2014 -0800

    shell: Only assign focus on touch if there is a touch focus surface

    Similar to 7c4f6cc145daf66cd9624f1e70dd5278241f5858, if we don't have
    a background image from the desktop-shell client or the touch point
    for some other reason doesn't hit a surface we trigger a
    segfault as we try to deref the seat->touch->focus NULL pointer.

    For touch, another problem could be fudgey calibration that ends up
    giving touch coordinates outside the output space.

    https://bugs.freedesktop.org/show_bug.cgi?id=72839

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-bugs/attachments/20140103/d9011f55/attachment-0001.html>