From e463b49166841f71c6ee9b8922ae5dc8973174ce Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Guido=20G=C3=BCnther?= Date: Wed, 11 Jul 2018 17:29:41 +0200 Subject: [PATCH] x11: Check if xcb_configure_window worked So far we did not check for any errors --- backend/x11/output.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/backend/x11/output.c b/backend/x11/output.c index 9f8918ab..151807dd 100644 --- a/backend/x11/output.c +++ b/backend/x11/output.c @@ -47,8 +47,16 @@ static bool output_set_custom_mode(struct wlr_output *wlr_output, output_set_refresh(&output->wlr_output, refresh); const uint32_t values[] = { width, height }; - xcb_configure_window(x11->xcb_conn, output->win, + xcb_void_cookie_t cookie = xcb_configure_window_checked(x11->xcb_conn, output->win, XCB_CONFIG_WINDOW_WIDTH | XCB_CONFIG_WINDOW_HEIGHT, values); + + xcb_generic_error_t *error; + if ((error = xcb_request_check(x11->xcb_conn, cookie))) { + wlr_log(WLR_ERROR, "Could not set window size to %dx%d\n", width, height); + free(error); + return false; + } + return true; }