Don't segfault when there's no target in If
authorMikael Magnusson <mikachu@gmail.com>
Tue, 17 Sep 2013 16:47:45 +0000 (18:47 +0200)
committerMikael Magnusson <mikachu@gmail.com>
Thu, 10 Apr 2014 20:29:01 +0000 (22:29 +0200)
openbox/actions/if.c

index a083d48..67d067e 100644 (file)
@@ -327,7 +327,10 @@ static gboolean run_func_if(ObActionsData *data, gpointer options)
         }
 
         /* If there's no client to query, then false. */
-        is_true &= query_target != NULL;
+        if (!query_target) {
+            is_true = FALSE;
+            break;
+        }
 
         if (q->shaded_on)
             is_true &= query_target->shaded;