client: Ignore new requests if display has a fatal error

Once there has been a fatal display error, any new object requests
potentially rely on invalid state. (For example, a failure to read
from the compositor could hide a important event.) The safest way to
handle the new requests is not to make them.

Proxies produced by the request are still created, to ensure that
any code using the library does not crash from an unexpected NULL
pointer.

Signed-off-by: Manuel Stoeckl <code@mstoeckl.com>

diff --git a/src/wayland-client.c b/src/wayland-client.c
index ed50281..7c93c7b 100644 (file)
--- a/src/wayland-client.c
+++ b/src/wayland-client.c
@@ -740,6 +740,10 @@ wl_proxy_marshal_array_constructor_versioned(struct wl_proxy *proxy,
                        goto err_unlock;
        }
 
+       if (proxy->display->last_error) {
+               goto err_unlock;
+       }
+
        closure = wl_closure_marshal(&proxy->object, opcode, args, message);
        if (closure == NULL)
                wl_abort("Error marshalling request: %s\n", strerror(errno));