[PATCH] dnd: fix segfault on grabbing

[Philipp Brüschweiler]

This fixes https://bugs.freedesktop.org/show_bug.cgi?id=50487

Cheers,
Philipp

On Mon, Aug 13, 2012 at 8:04 PM, Philipp Brüschweiler <blei42 at gmail.com> wrote:
> Some cursor themes don't include a "grabbing" icon, causing a segfault.
> This patch fixes this by just reverting to the stardard cursor instead.
> ---
>  clients/dnd.c | 7 +++++++
>  1 Datei geändert, 7 Zeilen hinzugefügt(+)
>
> diff --git a/clients/dnd.c b/clients/dnd.c
> index 6f55e95..44c1541 100644
> --- a/clients/dnd.c
> +++ b/clients/dnd.c
> @@ -20,6 +20,7 @@
>   * OF THIS SOFTWARE.
>   */
>
> +#include <assert.h>
>  #include <stdint.h>
>  #include <stdio.h>
>  #include <stdlib.h>
> @@ -327,6 +328,12 @@ create_drag_cursor(struct dnd_drag *dnd_drag,
>         cairo_t *cr;
>
>         pointer = display_get_pointer_image(dnd->display, CURSOR_DRAGGING);
> +       if (!pointer) {
> +               fprintf(stderr, "WARNING: grabbing cursor image not found\n");
> +               pointer = display_get_pointer_image(dnd->display,
> +                                                   CURSOR_LEFT_PTR);
> +               assert(pointer && "no cursor image found");
> +       }
>
>         rectangle.width = item_width + 2 * pointer->width;
>         rectangle.height = item_height + 2 * pointer->height;
> --
> 1.7.11.4
>