We can sizeof the struct type instead of declaring a pointer and
taking the size of what it points to.

Signed-off-by: Derek Foreman <der...@osg.samsung.com>
---
 src/connection.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/connection.c b/src/connection.c
index e60ad75..d029914 100644
--- a/src/connection.c
+++ b/src/connection.c
@@ -632,7 +632,7 @@ wl_connection_demarshal(struct wl_connection *connection,
        const char *signature;
        struct argument_details arg;
        struct wl_closure *closure;
-       struct wl_array *array, *array_extra;
+       struct wl_array *array_extra;
 
        count = arg_count_for_signature(message->signature);
        if (count > WL_CLOSURE_MAX_ARGS) {
@@ -643,7 +643,8 @@ wl_connection_demarshal(struct wl_connection *connection,
        }
 
        num_arrays = wl_message_count_arrays(message);
-       closure = malloc(sizeof *closure + size + num_arrays * sizeof *array);
+       closure = malloc(sizeof *closure + size +
+                        num_arrays * sizeof(struct wl_array));
        if (closure == NULL) {
                errno = ENOMEM;
                wl_connection_consume(connection, size);
-- 
2.15.0

_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to