Handle NULL from output_get_active_workspace

This modifies the places where output_get_active_workspace is called to
handle a NULL result. Some places already handled it and did not need a
change, some just have guard off code blocks, others return errors, and
some have sway_asserts since the case should never happen. A lot of this
is probably just safety precautions since they probably will never be
called when `output_get_active_workspace` is not fully configured with a
workspace.

1 files changed, 3 insertions, 0 deletions

diff --git a/sway/commands/focus.c b/sway/commands/focus.c
index 79b2f551..25df5130 100644
--- a/sway/commands/focus.c
+++ b/sway/commands/focus.c
@@ -37,6 +37,9 @@ static struct sway_node *get_node_in_output_direction(
                 struct sway_output *output, enum wlr_direction dir) {
         struct sway_seat *seat = config->handler_context.seat;
         struct sway_workspace *ws = output_get_active_workspace(output);
+        if (!sway_assert(ws, "Expected output to have a workspace")) {
+                return NULL;
+        }
         if (ws->fullscreen) {
                 return seat_get_focus_inactive(seat, &ws->fullscreen->node);
         }