server: Handle OOM properly when we fail to allocate a send closure

If we can't allocate a closure, don't just silently continue.  Set
client->error so we shut down the client when we're done processing events.

diff --git a/src/wayland-server.c b/src/wayland-server.c
index 0a6e112..b74f87a 100644 (file)
--- a/src/wayland-server.c
+++ b/src/wayland-server.c
@@ -136,8 +136,10 @@ wl_resource_post_event(struct wl_resource *resource, uint32_t opcode, ...)
                                      &object->interface->events[opcode]);
        va_end(ap);
 
-       if (closure == NULL)
+       if (closure == NULL) {
+               resource->client->error = 1
                return;
+       }
 
        if (wl_closure_send(closure, resource->client->connection))
                wl_event_loop_add_idle(resource->client->display->loop,
@@ -162,8 +164,10 @@ wl_resource_queue_event(struct wl_resource *resource, uint32_t opcode, ...)
                                      &object->interface->events[opcode]);
        va_end(ap);
 
-       if (closure == NULL)
+       if (closure == NULL) {
+               resource->client->error = 1
                return;
+       }
 
        if (wl_closure_queue(closure, resource->client->connection))
                wl_event_loop_add_idle(resource->client->display->loop,