[Wayland-bugs] [Bug 73950] weston aborts on touching the secondary output when primary output (touch device) is unplugged

[bugzilla-daemon at freedesktop.org]

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

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

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

--- Comment #4 from Kristian Høgsberg <krh at bitplanet.net> ---
commit ae826cead7caa46e03984955d38631ad0ea0c8fd
Author: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira at intel.com>
Date:   Thu Apr 24 15:11:16 2014 +0300

    evdev: Fix assertion error for unplugged output with paired touchscreen

    If the output a touchscreen is paired to is unplugged, events coming
    from it should be ignored. Commit 17bccaed introduced logic for that
    in evdev_flush_pending_damage(). However, the break statements it
    introduced would cause the assertion after the switch statement to
    fail.

    That function has the odd behavior that goto's are used to skip the
    assertion after the switch statement and jump to the hunk of code that
    marks the event as processed. Only in the case where the event type has
    an invalid value the assertion should trigger. So this patch fixes the
    problem by moving the assertion into the default case of the switch
    and replacing the goto statements with break ones.

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

-- 
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/20140425/8b71ae0b/attachment.html>