From 2176c63856df4e411df0ba3c5eee43f69e977b6e Mon Sep 17 00:00:00 2001 From: Simon Ser Date: Tue, 19 May 2020 17:28:23 +0200 Subject: [PATCH] xwayland: add option to disable WM --- include/wlr/xwayland.h | 8 ++++++- xwayland/server.c | 47 ++++++++++++++++++++++++++++++++---------- xwayland/xwayland.c | 6 +++++- 3 files changed, 48 insertions(+), 13 deletions(-) diff --git a/include/wlr/xwayland.h b/include/wlr/xwayland.h index c4c5842c..03201718 100644 --- a/include/wlr/xwayland.h +++ b/include/wlr/xwayland.h @@ -34,6 +34,7 @@ struct wlr_xwayland_server { int x_fd[2]; struct wl_event_source *x_fd_read_event[2]; bool lazy; + bool enable_wm; struct wl_display *wl_display; @@ -48,6 +49,11 @@ struct wlr_xwayland_server { void *data; }; +struct wlr_xwayland_server_options { + bool lazy; + bool enable_wm; +}; + struct wlr_xwayland_server_ready_event { struct wlr_xwayland_server *server; int wm_fd; @@ -216,7 +222,7 @@ struct wlr_xwayland_resize_event { }; struct wlr_xwayland_server *wlr_xwayland_server_create( - struct wl_display *display, bool lazy); + struct wl_display *display, struct wlr_xwayland_server_options *options); void wlr_xwayland_server_destroy(struct wlr_xwayland_server *server); /** Create an Xwayland server and XWM. diff --git a/xwayland/server.c b/xwayland/server.c index 0c76a389..c8c5413a 100644 --- a/xwayland/server.c +++ b/xwayland/server.c @@ -47,8 +47,12 @@ static int fill_arg(char ***argv, const char *fmt, ...) { noreturn static void exec_xwayland(struct wlr_xwayland_server *server) { if (!set_cloexec(server->x_fd[0], false) || !set_cloexec(server->x_fd[1], false) || - !set_cloexec(server->wm_fd[1], false) || !set_cloexec(server->wl_fd[1], false)) { + wlr_log(WLR_ERROR, "Failed to unset CLOEXEC on FD"); + _exit(EXIT_FAILURE); + } + if (server->enable_wm && !set_cloexec(server->wm_fd[1], false)) { + wlr_log(WLR_ERROR, "Failed to unset CLOEXEC on FD"); _exit(EXIT_FAILURE); } @@ -67,11 +71,19 @@ noreturn static void exec_xwayland(struct wlr_xwayland_server *server) { if (fill_arg(&cur_arg, ":%d", server->display) < 0 || fill_arg(&cur_arg, "%d", server->x_fd[0]) < 0 || - fill_arg(&cur_arg, "%d", server->x_fd[1]) < 0 || - fill_arg(&cur_arg, "%d", server->wm_fd[1]) < 0) { + fill_arg(&cur_arg, "%d", server->x_fd[1]) < 0) { wlr_log_errno(WLR_ERROR, "alloc/print failure"); _exit(EXIT_FAILURE); } + if (server->enable_wm) { + if (fill_arg(&cur_arg, "%d", server->wm_fd[1]) < 0) { + wlr_log_errno(WLR_ERROR, "alloc/print failure"); + _exit(EXIT_FAILURE); + } + } else { + cur_arg++; + *cur_arg = NULL; + } char wayland_socket_str[16]; snprintf(wayland_socket_str, sizeof(wayland_socket_str), "%d", server->wl_fd[1]); @@ -249,19 +261,30 @@ static bool server_start_display(struct wlr_xwayland_server *server, } static bool server_start(struct wlr_xwayland_server *server) { - if (socketpair(AF_UNIX, SOCK_STREAM, 0, server->wl_fd) != 0 || - socketpair(AF_UNIX, SOCK_STREAM, 0, server->wm_fd) != 0) { + if (socketpair(AF_UNIX, SOCK_STREAM, 0, server->wl_fd) != 0) { wlr_log_errno(WLR_ERROR, "socketpair failed"); server_finish_process(server); return false; } if (!set_cloexec(server->wl_fd[0], true) || - !set_cloexec(server->wl_fd[1], true) || - !set_cloexec(server->wm_fd[0], true) || - !set_cloexec(server->wm_fd[1], true)) { + !set_cloexec(server->wl_fd[1], true)) { + wlr_log(WLR_ERROR, "Failed to set O_CLOEXEC on socket"); server_finish_process(server); return false; } + if (server->enable_wm) { + if (socketpair(AF_UNIX, SOCK_STREAM, 0, server->wm_fd) != 0) { + wlr_log_errno(WLR_ERROR, "socketpair failed"); + server_finish_process(server); + return false; + } + if (!set_cloexec(server->wm_fd[0], true) || + !set_cloexec(server->wm_fd[1], true)) { + wlr_log(WLR_ERROR, "Failed to set O_CLOEXEC on socket"); + server_finish_process(server); + return false; + } + } server->server_start = time(NULL); @@ -319,7 +342,7 @@ static bool server_start(struct wlr_xwayland_server *server) { /* close child fds */ /* remain managing x sockets for lazy start */ close(server->wl_fd[1]); - close(server->wm_fd[1]); + safe_close(server->wm_fd[1]); server->wl_fd[1] = server->wm_fd[1] = -1; return true; @@ -359,7 +382,8 @@ void wlr_xwayland_server_destroy(struct wlr_xwayland_server *server) { } struct wlr_xwayland_server *wlr_xwayland_server_create( - struct wl_display *wl_display, bool lazy) { + struct wl_display *wl_display, + struct wlr_xwayland_server_options *options) { struct wlr_xwayland_server *server = calloc(1, sizeof(struct wlr_xwayland_server)); if (!server) { @@ -367,7 +391,8 @@ struct wlr_xwayland_server *wlr_xwayland_server_create( } server->wl_display = wl_display; - server->lazy = lazy; + server->lazy = options->lazy; + server->enable_wm = options->enable_wm; server->x_fd[0] = server->x_fd[1] = -1; server->wl_fd[0] = server->wl_fd[1] = -1; diff --git a/xwayland/xwayland.c b/xwayland/xwayland.c index ae86d541..a28cd402 100644 --- a/xwayland/xwayland.c +++ b/xwayland/xwayland.c @@ -85,7 +85,11 @@ struct wlr_xwayland *wlr_xwayland_create(struct wl_display *wl_display, wl_signal_init(&xwayland->events.new_surface); wl_signal_init(&xwayland->events.ready); - xwayland->server = wlr_xwayland_server_create(wl_display, lazy); + struct wlr_xwayland_server_options options = { + .lazy = lazy, + .enable_wm = true, + }; + xwayland->server = wlr_xwayland_server_create(wl_display, &options); if (xwayland->server == NULL) { free(xwayland->server); return NULL;