From 78245a3998ff467e111d9c4e7b05d7069719c125 Mon Sep 17 00:00:00 2001 From: emersion Date: Tue, 31 Oct 2017 11:29:53 +0100 Subject: [PATCH 1/2] Check allocation errors in wlr_output_layout --- types/wlr_output_layout.c | 67 ++++++++++++++++++++++++--------------- 1 file changed, 42 insertions(+), 25 deletions(-) diff --git a/types/wlr_output_layout.c b/types/wlr_output_layout.c index 8a50e331..d3f41766 100644 --- a/types/wlr_output_layout.c +++ b/types/wlr_output_layout.c @@ -8,14 +8,14 @@ #include struct wlr_output_layout_state { - struct wlr_box *_box; + struct wlr_box box; }; struct wlr_output_layout_output_state { struct wlr_output_layout *layout; struct wlr_output_layout_output *l_output; - struct wlr_box *_box; + struct wlr_box box; bool auto_configured; struct wl_listener resolution; @@ -25,8 +25,14 @@ struct wlr_output_layout_output_state { struct wlr_output_layout *wlr_output_layout_create() { struct wlr_output_layout *layout = calloc(1, sizeof(struct wlr_output_layout)); + if (layout == NULL) { + return NULL; + } layout->state = calloc(1, sizeof(struct wlr_output_layout_state)); - layout->state->_box = calloc(1, sizeof(struct wlr_box)); + if (layout->state == NULL) { + free(layout); + return NULL; + } wl_list_init(&layout->outputs); wl_signal_init(&layout->events.change); @@ -40,7 +46,6 @@ static void wlr_output_layout_output_destroy( wl_list_remove(&l_output->state->resolution.link); wl_list_remove(&l_output->state->output_destroy.link); wl_list_remove(&l_output->link); - free(l_output->state->_box); free(l_output->state); free(l_output); } @@ -52,25 +57,24 @@ void wlr_output_layout_destroy(struct wlr_output_layout *layout) { wl_signal_emit(&layout->events.destroy, layout); - struct wlr_output_layout_output *_output, *temp = NULL; - wl_list_for_each_safe(_output, temp, &layout->outputs, link) { - wlr_output_layout_output_destroy(_output); + struct wlr_output_layout_output *output, *temp = NULL; + wl_list_for_each_safe(output, temp, &layout->outputs, link) { + wlr_output_layout_output_destroy(output); } - free(layout->state->_box); free(layout->state); free(layout); } static struct wlr_box *wlr_output_layout_output_get_box( struct wlr_output_layout_output *l_output) { - l_output->state->_box->x = l_output->x; - l_output->state->_box->y = l_output->y; + l_output->state->box.x = l_output->x; + l_output->state->box.y = l_output->y; int width, height; wlr_output_effective_resolution(l_output->output, &width, &height); - l_output->state->_box->width = width; - l_output->state->_box->height = height; - return l_output->state->_box; + l_output->state->box.width = width; + l_output->state->box.height = height; + return &l_output->state->box; } /** @@ -138,18 +142,24 @@ static void handle_output_destroy(struct wl_listener *listener, void *data) { static struct wlr_output_layout_output *wlr_output_layout_output_create( struct wlr_output_layout *layout, struct wlr_output *output) { - struct wlr_output_layout_output *l_output; - l_output= calloc(1, sizeof(struct wlr_output_layout_output)); + struct wlr_output_layout_output *l_output = + calloc(1, sizeof(struct wlr_output_layout_output)); + if (l_output == NULL) { + return NULL; + } l_output->state = calloc(1, sizeof(struct wlr_output_layout_output_state)); + if (l_output->state == NULL) { + free(l_output); + return NULL; + } l_output->state->l_output = l_output; - l_output->state->_box = calloc(1, sizeof(struct wlr_box)); l_output->state->layout = layout; l_output->output = output; + wl_list_insert(&layout->outputs, &l_output->link); wl_signal_add(&output->events.resolution, &l_output->state->resolution); l_output->state->resolution.notify = handle_output_resolution; - wl_signal_add(&output->events.destroy, &l_output->state->output_destroy); l_output->state->output_destroy.notify = handle_output_destroy; @@ -162,6 +172,10 @@ void wlr_output_layout_add(struct wlr_output_layout *layout, wlr_output_layout_get(layout, output); if (!l_output) { l_output = wlr_output_layout_output_create(layout, output); + if (!l_output) { + wlr_log(L_ERROR, "Failed to create wlr_output_layout_output"); + return; + } } l_output->x = x; l_output->y = y; @@ -238,8 +252,8 @@ void wlr_output_layout_move(struct wlr_output_layout *layout, void wlr_output_layout_remove(struct wlr_output_layout *layout, struct wlr_output *output) { - struct wlr_output_layout_output *l_output; - l_output= wlr_output_layout_get(layout, output); + struct wlr_output_layout_output *l_output = + wlr_output_layout_get(layout, output); if (l_output) { wlr_output_layout_output_destroy(l_output); wlr_output_layout_reconfigure(layout); @@ -324,12 +338,12 @@ struct wlr_box *wlr_output_layout_get_box( } } - layout->state->_box->x = min_x; - layout->state->_box->y = min_y; - layout->state->_box->width = max_x - min_x; - layout->state->_box->height = max_y - min_y; + layout->state->box.x = min_x; + layout->state->box.y = min_y; + layout->state->box.width = max_x - min_x; + layout->state->box.height = max_y - min_y; - return layout->state->_box; + return &layout->state->box; } // not reached @@ -339,9 +353,12 @@ void wlr_output_layout_add_auto(struct wlr_output_layout *layout, struct wlr_output *output) { struct wlr_output_layout_output *l_output = wlr_output_layout_get(layout, output); - if (!l_output) { l_output = wlr_output_layout_output_create(layout, output); + if (!l_output) { + wlr_log(L_ERROR, "Failed to create wlr_output_layout_output"); + return; + } } l_output->state->auto_configured = true; From 88f85f74773b7b5f4a35a5f9f53c336078d773af Mon Sep 17 00:00:00 2001 From: emersion Date: Tue, 31 Oct 2017 12:01:49 +0100 Subject: [PATCH 2/2] Fix code style issues --- types/wlr_output_layout.c | 30 +++++++++++++++--------------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/types/wlr_output_layout.c b/types/wlr_output_layout.c index d3f41766..817a03f6 100644 --- a/types/wlr_output_layout.c +++ b/types/wlr_output_layout.c @@ -8,14 +8,14 @@ #include struct wlr_output_layout_state { - struct wlr_box box; + struct wlr_box _box; // should never be read directly, use the getter }; struct wlr_output_layout_output_state { struct wlr_output_layout *layout; struct wlr_output_layout_output *l_output; - struct wlr_box box; + struct wlr_box _box; // should never be read directly, use the getter bool auto_configured; struct wl_listener resolution; @@ -57,9 +57,9 @@ void wlr_output_layout_destroy(struct wlr_output_layout *layout) { wl_signal_emit(&layout->events.destroy, layout); - struct wlr_output_layout_output *output, *temp = NULL; - wl_list_for_each_safe(output, temp, &layout->outputs, link) { - wlr_output_layout_output_destroy(output); + struct wlr_output_layout_output *l_output, *temp = NULL; + wl_list_for_each_safe(l_output, temp, &layout->outputs, link) { + wlr_output_layout_output_destroy(l_output); } free(layout->state); @@ -68,13 +68,13 @@ void wlr_output_layout_destroy(struct wlr_output_layout *layout) { static struct wlr_box *wlr_output_layout_output_get_box( struct wlr_output_layout_output *l_output) { - l_output->state->box.x = l_output->x; - l_output->state->box.y = l_output->y; + l_output->state->_box.x = l_output->x; + l_output->state->_box.y = l_output->y; int width, height; wlr_output_effective_resolution(l_output->output, &width, &height); - l_output->state->box.width = width; - l_output->state->box.height = height; - return &l_output->state->box; + l_output->state->_box.width = width; + l_output->state->_box.height = height; + return &l_output->state->_box; } /** @@ -338,12 +338,12 @@ struct wlr_box *wlr_output_layout_get_box( } } - layout->state->box.x = min_x; - layout->state->box.y = min_y; - layout->state->box.width = max_x - min_x; - layout->state->box.height = max_y - min_y; + layout->state->_box.x = min_x; + layout->state->_box.y = min_y; + layout->state->_box.width = max_x - min_x; + layout->state->_box.height = max_y - min_y; - return &layout->state->box; + return &layout->state->_box; } // not reached