ivi-shell: use x*alloc for small allocations

Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
This commit is contained in:
Michael Olbrich 2023-02-06 16:34:35 +01:00
parent c7b9625775
commit c4fab1633b
2 changed files with 15 additions and 62 deletions

View File

@ -71,6 +71,7 @@
#include "shared/helpers.h"
#include "shared/os-compatibility.h"
#include "shared/signal.h"
#include "shared/xalloc.h"
#define max(a, b) ((a) > (b) ? (a) : (b))
@ -167,11 +168,7 @@ ivi_view_create(struct ivi_layout_layer *ivilayer,
{
struct ivi_layout_view *ivi_view;
ivi_view = calloc(1, sizeof *ivi_view);
if (ivi_view == NULL) {
weston_log("fails to allocate memory\n");
return NULL;
}
ivi_view = xzalloc(sizeof *ivi_view);
if (weston_surface_is_desktop_surface(ivisurf->surface)) {
ivi_view->view = weston_desktop_surface_create_view(
@ -305,11 +302,7 @@ add_screen(struct weston_output *output)
struct ivi_layout *layout = get_instance();
struct ivi_layout_screen *iviscrn = NULL;
iviscrn = calloc(1, sizeof *iviscrn);
if (iviscrn == NULL) {
weston_log("fails to allocate memory\n");
return;
}
iviscrn = xzalloc(sizeof *iviscrn);
iviscrn->layout = layout;
iviscrn->output = output;
@ -1137,11 +1130,7 @@ ivi_layout_get_screens_under_layer(struct ivi_layout_layer *ivilayer,
if (length != 0) {
/* the Array must be free by module which called this function */
*ppArray = calloc(length, sizeof(struct weston_output *));
if (*ppArray == NULL) {
weston_log("fails to allocate memory\n");
return IVI_FAILED;
}
*ppArray = xcalloc(length, sizeof(struct weston_output *));
(*ppArray)[n++] = ivilayer->on_screen->output;
}
@ -1168,11 +1157,7 @@ ivi_layout_get_layers(int32_t *pLength, struct ivi_layout_layer ***ppArray)
if (length != 0) {
/* the Array must be freed by module which called this function */
*ppArray = calloc(length, sizeof(struct ivi_layout_layer *));
if (*ppArray == NULL) {
weston_log("fails to allocate memory\n");
return IVI_FAILED;
}
*ppArray = xcalloc(length, sizeof(struct ivi_layout_layer *));
wl_list_for_each(ivilayer, &layout->layer_list, link) {
(*ppArray)[n++] = ivilayer;
@ -1204,11 +1189,7 @@ ivi_layout_get_layers_on_screen(struct weston_output *output,
if (length != 0) {
/* the Array must be freed by module which called this function */
*ppArray = calloc(length, sizeof(struct ivi_layout_layer *));
if (*ppArray == NULL) {
weston_log("fails to allocate memory\n");
return IVI_FAILED;
}
*ppArray = xcalloc(length, sizeof(struct ivi_layout_layer *));
wl_list_for_each(ivilayer, &iviscrn->order.layer_list, order.link) {
(*ppArray)[n++] = ivilayer;
@ -1237,11 +1218,7 @@ ivi_layout_get_layers_under_surface(struct ivi_layout_surface *ivisurf,
if (!wl_list_empty(&ivisurf->view_list)) {
/* the Array must be free by module which called this function */
length = wl_list_length(&ivisurf->view_list);
*ppArray = calloc(length, sizeof(struct ivi_layout_layer *));
if (*ppArray == NULL) {
weston_log("fails to allocate memory\n");
return IVI_FAILED;
}
*ppArray = xcalloc(length, sizeof(struct ivi_layout_layer *));
wl_list_for_each_reverse(ivi_view, &ivisurf->view_list, surf_link) {
if (ivi_view_is_rendered(ivi_view))
@ -1278,11 +1255,7 @@ ivi_layout_get_surfaces(int32_t *pLength, struct ivi_layout_surface ***ppArray)
if (length != 0) {
/* the Array must be freed by module which called this function */
*ppArray = calloc(length, sizeof(struct ivi_layout_surface *));
if (*ppArray == NULL) {
weston_log("fails to allocate memory\n");
return IVI_FAILED;
}
*ppArray = xcalloc(length, sizeof(struct ivi_layout_surface *));
wl_list_for_each(ivisurf, &layout->surface_list, link) {
(*ppArray)[n++] = ivisurf;
@ -1312,11 +1285,7 @@ ivi_layout_get_surfaces_on_layer(struct ivi_layout_layer *ivilayer,
if (length != 0) {
/* the Array must be freed by module which called this function */
*ppArray = calloc(length, sizeof(struct ivi_layout_surface *));
if (*ppArray == NULL) {
weston_log("fails to allocate memory\n");
return IVI_FAILED;
}
*ppArray = xcalloc(length, sizeof(struct ivi_layout_surface *));
wl_list_for_each(ivi_view, &ivilayer->order.view_list, order_link) {
(*ppArray)[n++] = ivi_view->ivisurf;
@ -1342,11 +1311,7 @@ ivi_layout_layer_create_with_dimension(uint32_t id_layer,
return ivilayer;
}
ivilayer = calloc(1, sizeof *ivilayer);
if (ivilayer == NULL) {
weston_log("fails to allocate memory\n");
return NULL;
}
ivilayer = xzalloc(sizeof *ivilayer);
ivilayer->ref_count = 1;
wl_signal_init(&ivilayer->property_changed);
@ -1996,11 +1961,7 @@ surface_create(struct weston_surface *wl_surface, uint32_t id_surface)
return NULL;
}
ivisurf = calloc(1, sizeof *ivisurf);
if (ivisurf == NULL) {
weston_log("fails to allocate memory\n");
return NULL;
}
ivisurf = xzalloc(sizeof *ivisurf);
wl_signal_init(&ivisurf->property_changed);
ivisurf->id_surface = id_surface;

View File

@ -47,6 +47,7 @@
#include "ivi-layout-private.h"
#include "ivi-layout-shell.h"
#include "shared/helpers.h"
#include "shared/xalloc.h"
#include "compositor/weston.h"
/* Representation of ivi_surface protocol object. */
@ -267,11 +268,7 @@ application_surface_create(struct wl_client *client,
layout_surface->weston_desktop_surface = NULL;
ivisurf = zalloc(sizeof *ivisurf);
if (ivisurf == NULL) {
wl_resource_post_no_memory(resource);
return;
}
ivisurf = xzalloc(sizeof *ivisurf);
wl_list_init(&ivisurf->link);
wl_list_insert(&shell->ivi_surface_list, &ivisurf->link);
@ -526,10 +523,7 @@ desktop_surface_added(struct weston_desktop_surface *surface,
return;
}
ivisurf = zalloc(sizeof *ivisurf);
if (!ivisurf) {
return;
}
ivisurf = xzalloc(sizeof *ivisurf);
ivisurf->shell = shell;
ivisurf->id_surface = IVI_INVALID_ID;
@ -696,9 +690,7 @@ wet_shell_init(struct weston_compositor *compositor,
{
struct ivi_shell *shell;
shell = zalloc(sizeof *shell);
if (shell == NULL)
return -1;
shell = xzalloc(sizeof *shell);
if (!weston_compositor_add_destroy_listener_once(compositor,
&shell->destroy_listener,