From 0b17b8ec01116f64edb42d9864b2283022b342cf Mon Sep 17 00:00:00 2001 From: Rob Bradford Date: Thu, 5 Jan 2012 15:30:00 +0000 Subject: [PATCH] compositor: Don't try and destroy old pointer surface if it empty If attach was called on wl_input_device with a nil buffer twice then the second call would cause a segfault. --- src/compositor.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/compositor.c b/src/compositor.c index 819cb5d9..26279872 100644 --- a/src/compositor.c +++ b/src/compositor.c @@ -1609,8 +1609,10 @@ input_device_attach(struct wl_client *client, weston_surface_damage_below(device->sprite); if (!buffer_resource) { - destroy_surface(&device->sprite->surface.resource); - device->sprite = NULL; + if (device->sprite) { + destroy_surface(&device->sprite->surface.resource); + device->sprite = NULL; + } return; }