[PATCH][weston] ivi-shell: initialize all fields in create_layout_transition

Bill Spitzak spitzak at gmail.com
Mon Sep 21 09:33:29 PDT 2015


On Sat, Sep 19, 2015 at 5:47 PM, John-John Tedro <johnjohn.tedro at gmail.com>
wrote:

> create_layout_transition does not initialize is_transition_func.
>
> This patch initializes it to NULL.
>
> Signed-off-by: John-John Tedro <johnjohn.tedro at gmail.com>
> ---
>  ivi-shell/ivi-layout-transition.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/ivi-shell/ivi-layout-transition.c
> b/ivi-shell/ivi-layout-transition.c
> index d12a8f4..d45b332 100644
> --- a/ivi-shell/ivi-layout-transition.c
> +++ b/ivi-shell/ivi-layout-transition.c
> @@ -251,6 +251,7 @@ create_layout_transition(void)
>
>         transition->is_done = 0;
>
> +       transition->is_transition_func = NULL;
>         transition->private_data = NULL;
>         transition->user_data = NULL;
>

Maybe this should be using xzalloc, that seems to be the standard method in
Weston
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20150921/6665dab3/attachment.html>


More information about the wayland-devel mailing list