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/ipc-json.c b/sway/ipc-json.c
index 6d36c727..a2ab2bba 100644
--- a/sway/ipc-json.c
+++ b/sway/ipc-json.c
@@ -198,6 +198,9 @@ static void ipc_json_describe_output(struct sway_output *output,
                         ipc_json_output_transform_description(wlr_output->transform)));
 
         struct sway_workspace *ws = output_get_active_workspace(output);
+        if (!sway_assert(ws, "Expected output to have a workspace")) {
+                return;
+        }
         json_object_object_add(object, "current_workspace",
                         json_object_new_string(ws->name));