fix a few warnings/places where the clang static analyzer complains

[i3/i3] / src / con.c

diff --git a/src/con.c b/src/con.c
index cb4d080501607e0911249270c4fa01727c792407..01305f1a5744ed2c894b6124e1b5f993b58f1bc9 100644 (file)
--- a/src/con.c
+++ b/src/con.c
@@ -1260,6 +1260,9 @@ bool con_fullscreen_permits_focusing(Con *con) {
     while (fs && fs->fullscreen_mode == CF_NONE)
         fs = fs->parent;
 
+    /* fs must be non-NULL since the workspace con doesn’t have CF_NONE and
+     * there always has to be a workspace con in the hierarchy. */
+    assert(fs != NULL);
     /* The most common case is we hit the workspace level. In this
      * situation, changing focus is also harmless. */
     assert(fs->fullscreen_mode != CF_NONE);