fix opening a floating view on the NOOP output

Since the NOOP output has no size, the minimum floating size is greater
than the workspace size for the NOOP output. In this case, the floater
gets centered in the output instead of the workspace. However, the
NOOP output is not part of the output layout and thus has a NULL box.
Attempting to access the properties of this box was causing a segfault.

This fixes the issue by just setting the floater's box to all zeroes
when mapping on the NOOP output. When the workspace gets moved from the
NOOP output to a new output, any floater whose width or height is zero
or has an x or y location outside of the output, gets passed to
`container_init_floating` again. This will then set the appropriate
size and centering. For any floater that has a valid size and location,
they are preserved.

1 files changed, 11 insertions, 4 deletions

diff --git a/sway/tree/container.c b/sway/tree/container.c
index 93cff7ff..f84ce360 100644
--- a/sway/tree/container.c
+++ b/sway/tree/container.c
@@ -651,6 +651,17 @@ void floating_calculate_constraints(int *min_width, int *max_width,
 
 void container_init_floating(struct sway_container *con) {
         struct sway_workspace *ws = con->workspace;
+        struct wlr_box *ob = wlr_output_layout_get_box(root->output_layout,
+                        ws->output->wlr_output);
+        if (!ob) {
+                // On NOOP output. Will be called again when moved to an output
+                con->x = 0;
+                con->y = 0;
+                con->width = 0;
+                con->height = 0;
+                return;
+        }
+
         int min_width, max_width, min_height, max_height;
         floating_calculate_constraints(&min_width, &max_width,
                         &min_height, &max_height);
@@ -659,8 +670,6 @@ void container_init_floating(struct sway_container *con) {
                 con->width = max_width;
                 con->height = max_height;
                 if (con->width > ws->width || con->height > ws->height) {
-                        struct wlr_box *ob = wlr_output_layout_get_box(root->output_layout,
-                                        ws->output->wlr_output);
                         con->x = ob->x + (ob->width - con->width) / 2;
                         con->y = ob->y + (ob->height - con->height) / 2;
                 } else {
@@ -675,8 +684,6 @@ void container_init_floating(struct sway_container *con) {
                         fmax(min_height, fmin(view->natural_height, max_height));
                 if (con->content_width > ws->width
                                 || con->content_height > ws->height) {
-                        struct wlr_box *ob = wlr_output_layout_get_box(root->output_layout,
-                                        ws->output->wlr_output);
                         con->content_x = ob->x + (ob->width - con->content_width) / 2;
                         con->content_y = ob->y + (ob->height - con->content_height) / 2;
                 } else {