<html> <head> <base href="https://bugzilla.gnome.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - [Wayland] Crash under gdk_wayland_window_attach_image()" href="https://bugzilla.gnome.org/show_bug.cgi?id=773274#c8">Comment # 8</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - [Wayland] Crash under gdk_wayland_window_attach_image()" href="https://bugzilla.gnome.org/show_bug.cgi?id=773274">bug 773274</a> from <a href="page.cgi?id=describeuser.html&login=ofourdan%40redhat.com" title="Olivier Fourdan <ofourdan@redhat.com>"> Olivier Fourdan</a> <pre>Created <a href="attachment.cgi?id=338945&action=diff" name="attach_338945" title="[PATCH] wayland: check valid pending cairo surface">attachment 338945</a> <a href="attachment.cgi?id=338945&action=edit" title="[PATCH] wayland: check valid pending cairo surface">[details]</a> <a href='review?bug=773274&attachment=338945'>[review]</a> [PATCH] wayland: check valid pending cairo surface drop_cairo_surfaces() will nullify the drop_cairo_surfaces() (to avoid reusing the buffer later), but gdk_window_impl_wayland_end_paint() doesn't check if the drop_cairo_surfaces is still valid prior to calling gdk_wayland_window_attach_image(). As gdk_wayland_window_attach_image() assumes there is a valid pending cairo surface, make sure the staging cairo surface is stil lvalid prior to attaching the image, just like it's done in gdk_wayland_window_show() as well.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are on the CC list for the bug.</li> </ul> </body> </html>