diff options
| author | Jente Hidskes <hjdskes@gmail.com> | 2019-01-24 13:48:26 +0100 |
|---|---|---|
| committer | Jente Hidskes <hjdskes@gmail.com> | 2019-01-24 15:18:28 +0100 |
| commit | 85d84a1a0444bf4ccc7a6a396f2c285540c170ef (patch) | |
| tree | 3318695da3d7b4ed44e77e996eed2c9cb339483f /backend/x11 | |
| parent | 4cb0697e576820774013f399c89fe85673e2f338 (diff) | |
backend/x11 & backend/wayland: make set_title NULL-safe
Set the default "wlroots - " title when the title argument to the
set_title functions is NULL. Otherwise, for at least the Wayland
backend, we'd crash because xdg_toplevel_set_title doesn't handle a NULL
pointer.
Diffstat (limited to 'backend/x11')
| -rw-r--r-- | backend/x11/output.c | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/backend/x11/output.c b/backend/x11/output.c index 9c55b154..05df7984 100644 --- a/backend/x11/output.c +++ b/backend/x11/output.c @@ -185,10 +185,7 @@ struct wlr_output *wlr_x11_output_create(struct wlr_backend *backend) { x11->atoms.wm_protocols, XCB_ATOM_ATOM, 32, 1, &x11->atoms.wm_delete_window); - char title[32]; - if (snprintf(title, sizeof(title), "wlroots - %s", wlr_output->name)) { - wlr_x11_output_set_title(wlr_output, title); - } + wlr_x11_output_set_title(wlr_output, NULL); xcb_map_window(x11->xcb, output->win); xcb_flush(x11->xcb); @@ -236,6 +233,14 @@ bool wlr_output_is_x11(struct wlr_output *wlr_output) { void wlr_x11_output_set_title(struct wlr_output *output, const char *title) { struct wlr_x11_output *x11_output = get_x11_output_from_output(output); + char wl_title[32]; + if (title == NULL) { + if (snprintf(wl_title, sizeof(wl_title), "wlroots - %s", output->name) <= 0) { + return; + } + title = wl_title; + } + xcb_change_property(x11_output->x11->xcb, XCB_PROP_MODE_REPLACE, x11_output->win, x11_output->x11->atoms.net_wm_name, x11_output->x11->atoms.utf8_string, 8, strlen(title), title); |