<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW --- - SIGSEGV in wl_resource_get_client during fullscreen client close animation" href="https://bugs.freedesktop.org/show_bug.cgi?id=78706#c4">Comment # 4</a> on <a class="bz_bug_link bz_status_NEW " title="NEW --- - SIGSEGV in wl_resource_get_client during fullscreen client close animation" href="https://bugs.freedesktop.org/show_bug.cgi?id=78706">bug 78706</a> from <a class="email" href="mailto:stu_dby@126.com" title="Boyan Ding <stu_dby@126.com>"> Boyan Ding</a> <pre>(In reply to <a href="show_bug.cgi?id=78706#c3">comment #3</a>) > Created <a href="attachment.cgi?id=105304" name="attach_105304" title="Stop crash when starting a touch during a fullscreen fade">attachment 105304</a> <a href="attachment.cgi?id=105304&action=edit" title="Stop crash when starting a touch during a fullscreen fade">[details]</a> <a href='page.cgi?id=splinter.html&bug=78706&attachment=105304'>[review]</a> [review] > Stop crash when starting a touch during a fullscreen fade > > This issue is still reproducible today - I think this patch fixes it, but is > only partially tested because I don't have a touchscreen (I just mashed up > libinput to pretend my mouse is one). > > If you could test and confirm this works, that would be great. Looks good, although I don't have a touchscreen to test either. One little question here: can we use if (find_resource_for_view(&seat->touch->resource_list, view)) instead of if (view) to avoid the problem just as weston_pointer_set_focus do it? If that can be used, the code will be much cleaner. I also suggest sending the patch to the mailing list for review.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>