fix for click on title_bar

- Check null pointer for swayc_parent_by_type
- Split lines to follow coding style

1 files changed, 12 insertions, 7 deletions

diff --git a/sway/handlers.c b/sway/handlers.c
index b6bc11c0..f37473aa 100644
--- a/sway/handlers.c
+++ b/sway/handlers.c
@@ -776,10 +776,13 @@ static bool handle_pointer_motion(wlc_handle handle, uint32_t time, const struct
         return EVENT_PASSTHROUGH;
 }
 
-bool swayc_border_check(swayc_t *c, const void *_origin) {
+static bool swayc_border_check(swayc_t *c, const void *_origin) {
         const struct wlc_point *origin = _origin;
-        if (origin->x >= c->title_bar_geometry.origin.x && origin->y >= c->title_bar_geometry.origin.y
-                && origin->x < (c->title_bar_geometry.origin.x + (int)c->title_bar_geometry.size.w) && origin->y < (c->title_bar_geometry.origin.y + (int)c->title_bar_geometry.size.h)) {
+        const struct wlc_geometry title_bar = c->title_bar_geometry;
+
+        if (origin->x >= title_bar.origin.x && origin->y >= title_bar.origin.y
+                && origin->x < title_bar.origin.x + (int32_t)title_bar.size.w
+                && origin->y < title_bar.origin.y + (int32_t)title_bar.size.h) {
                 return true;
         }
         return false;
@@ -849,10 +852,12 @@ static bool handle_pointer_button(wlc_handle view, uint32_t time, const struct w
         swayc_t *pointer = pointer_state.view;
         if (pointer) {
                 swayc_t *ws = swayc_parent_by_type(focused, C_WORKSPACE);
-                swayc_t *find = container_find(ws, &swayc_border_check, origin);
-                if (find != NULL) {
-                        set_focused_container(find);
-                        return EVENT_HANDLED;
+                if (ws != NULL) {
+                                swayc_t *find = container_find(ws, &swayc_border_check, origin);
+                                if (find != NULL) {
+                                        set_focused_container(find);
+                                        return EVENT_HANDLED;
+                                }
                 }
 
                 if (focused != pointer) {