server: do not send delete_id to a dead client

During client tear-down, all objects are destroyed in id order.
Therefore the display object is destroyed first.

If the destroy listeners of any object destroy another object by calling
wl_resoruce_destroy(), we try to send a delete_id event to the client.
This leads to a segmentation fault without a display object.

Signed-off-by: Pekka Paalanen <ppaalanen@gmail.com>

1 files changed, 5 insertions, 3 deletions

diff --git a/src/wayland-server.c b/src/wayland-server.c
index 9358eb4..421c3f0 100644
--- a/src/wayland-server.c
+++ b/src/wayland-server.c
@@ -348,9 +348,11 @@ wl_resource_destroy(struct wl_resource *resource, uint32_t time)
 	struct wl_client *client = resource->client;
 
 	if (resource->object.id < WL_SERVER_ID_START) {
-		wl_resource_queue_event(resource->client->display_resource,
-					WL_DISPLAY_DELETE_ID,
-					resource->object.id);
+		if (client->display_resource) {
+			wl_resource_queue_event(client->display_resource,
+						WL_DISPLAY_DELETE_ID,
+						resource->object.id);
+		}
 		wl_map_insert_at(&client->objects, resource->object.id, NULL);
 	} else {
 		wl_map_remove(&client->objects, resource->object.id);