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, 6 insertions, 1 deletions

diff --git a/sway/commands/titlebar_border_thickness.c b/sway/commands/titlebar_border_thickness.c
index 3c5e9ba1..7c27c163 100644
--- a/sway/commands/titlebar_border_thickness.c
+++ b/sway/commands/titlebar_border_thickness.c
@@ -21,7 +21,12 @@ struct cmd_results *cmd_titlebar_border_thickness(int argc, char **argv) {
 
         for (int i = 0; i < root->outputs->length; ++i) {
                 struct sway_output *output = root->outputs->items[i];
-                arrange_workspace(output_get_active_workspace(output));
+                struct sway_workspace *ws = output_get_active_workspace(output);
+                if (!sway_assert(ws, "Expected output to have a workspace")) {
+                        return cmd_results_new(CMD_FAILURE,
+                                        "Expected output to have a workspace");
+                }
+                arrange_workspace(ws);
                 output_damage_whole(output);
         }