Re: [PATCH] wayland: fix crash when /usr/share/wayland/left_ptr.png is not available

[Scott Moreau <oreaus gmail com>]

On Sat, May 5, 2012 at 7:37 PM, Shawn Landden <shawnlandden gmail com> wrote:

Signed-off-by: Shawn Landden <shawnlandden gmail com>
---
 gdk/wayland/gdkdevice-wayland.c |    3 +++
 1 file changed, 3 insertions(+)

diff --git a/gdk/wayland/gdkdevice-wayland.c b/gdk/wayland/gdkdevice-wayland.c
index 125563a..8de2ccf 100644
--- a/gdk/wayland/gdkdevice-wayland.c
+++ b/gdk/wayland/gdkdevice-wayland.c
@@ -164,6 +164,9 @@ gdk_device_core_set_window_cursor (GdkDevice *device,
      /* FIXME: Is this the best sensible default ? */
      cursor = _gdk_wayland_display_get_cursor_for_type (device->display,
                                                         GDK_LEFT_PTR);
+      /* above returns NULL if it can't find the specified cursor */
+      if (!cursor)
+        return;
    }

  buffer = _gdk_wayland_cursor_get_buffer(cursor, &x, &y);
--
1.7.9.5

Didn't you already submit this 2 days ago? http://lists.freedesktop.org/archives/wayland-devel/2012-May/003237.html and now again http://lists.freedesktop.org/archives/wayland-devel/2012-May/003271.html

Unless you are submitting a new change to your previous patch, you only need to submit it one time.



Thanks,

Scott