[Wayland-bugs] [Bug 92736] Weston crash when touch->focus null

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Tue Feb 2 18:38:54 UTC 2016


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

Bryce Harrington <b.harrington at samsung.com> changed:

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

--- Comment #3 from Bryce Harrington <b.harrington at samsung.com> ---
Thanks Guanghua, good find.  Looks like we were checking the pointer in the
function, but only after it was used.

I've pushed a slightly modified version of your patch to trunk.  Thanks again!

commit 2c40d1d30eabe46706822a8b5c94579aeba58e84
Author:     Bryce Harrington <bryce at bryceharrington.org>
AuthorDate: Tue Feb 2 10:18:48 2016 -0800
Commit:     Bryce Harrington <bryce at bryceharrington.org>
CommitDate: Tue Feb 2 10:36:44 2016 -0800

    input: Fix crash when touchscreen generates out of screen coordinate

    With change 61ed7b6b, global touch coordinates are being passed to the
    touch grab.  However, touch->grab is undefined in certain circumstances
    such as when the touch screen raises an axis X value larger than the
    maximum expected.  Move the check for this condition earlier, before our
    first use of the pointer.

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

    Reviewed-by: Bryce Harrington <bryce at osg.samsung.com>
    Reviewed-by: Derek Foreman <derekf at osg.samsung.com>

-- 
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/20160202/f99f2af8/attachment.html>


More information about the wayland-bugs mailing list