<div dir="ltr"><div class="gmail_extra"><div><div><div dir="ltr">On Wed, Sep 23, 2015 at 9:06 PM, Bryce Harrington <span dir="ltr"><<a href="mailto:bryce@osg.samsung.com" target="_blank">bryce@osg.samsung.com</a>></span> wrote:<br></div></div></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span>On Wed, Sep 23, 2015 at 10:33:21AM -0300, Lucas Tanure wrote:<br>
> Free transition in ivi_layout_transition_fade_layer if we fail to<br>
> allocate memory for data<br>
><br>
> Signed-off-by: Lucas Tanure <<a href="mailto:tanure@linux.com" target="_blank">tanure@linux.com</a>><br>
<br>
</span>Don't forget spaces after keywords.  I.e. "if (" instead of "if("<br>
<br>
I locally fixed that code style issue, as well as the missing semicolon<br>
on the third patch.  Then since the three patches all have the same<br>
title, and are each basically just one-liners, I squashed the whole<br>
collection into one commit.<br>
<br>
commit c8dcd16b01ea0f5ac8f32ee18374b2ca2694dd6c<br>
Author:     Lucas Tanure <<a href="mailto:tanure@linux.com" target="_blank">tanure@linux.com</a>><br>
AuthorDate: Wed Sep 23 10:33:<a href="tel:21%202015%20-0300" value="+552120150300" target="_blank">21 2015 -0300</a><br>
Commit:     Bryce Harrington <<a href="mailto:bryce@osg.samsung.com" target="_blank">bryce@osg.samsung.com</a>><br>
CommitDate: Wed Sep 23 17:02:<a href="tel:28%202015%20-0700" value="+552820150700" target="_blank">28 2015 -0700</a><br>
<br>
    ivi-layout-transition: Fix potential leak of memory pointed to by transition<br>
<br>
    Several fixes to handle invalid transition objects:<br>
<br>
    1. Free transition in ivi_layout_transition_fade_layer if we fail to<br>
       allocate memory for data<br>
<br>
    2. Check if transition is not null and if layout_transition_register was<br>
<span>       ok before return, if not, free transition<br>
<br>
</span>    3. Destroy transition if layout_transition_register was not ok, we can't<br>
<span>       just free transition, we need to call layout_transition_destroy in<br>
       order to free private_data from transition<br>
<br>
    Signed-off-by: Lucas Tanure <<a href="mailto:tanure@linux.com" target="_blank">tanure@linux.com</a>><br>
</span>    Reviewed-by: Bryce Harrington <<a href="mailto:bryce@osg.samsung.com" target="_blank">bryce@osg.samsung.com</a>><br>
<br>
And pushed to trunk:<br>
<br>
To ssh://<a href="http://git.freedesktop.org/git/wayland/weston" rel="noreferrer" target="_blank">git.freedesktop.org/git/wayland/weston</a><br>
   ae5df83..c8dcd16  master -> master<br>
<span><br>
<br>
> ---<br>
>  ivi-shell/ivi-layout-transition.c | 1 +<br>
>  1 file changed, 1 insertion(+)<br>
><br>
> diff --git a/ivi-shell/ivi-layout-transition.c b/ivi-shell/ivi-layout-transition.c<br>
> index 150e301..716dd70 100644<br>
> --- a/ivi-shell/ivi-layout-transition.c<br>
> +++ b/ivi-shell/ivi-layout-transition.c<br>
> @@ -850,6 +850,7 @@ ivi_layout_transition_fade_layer(<br>
>       data = malloc(sizeof(*data));<br>
>       if (data == NULL) {<br>
>               weston_log("%s: memory allocation fails\n", __func__);<br>
> +             free(transition);<br>
>               return;<br>
>       }<br>
><br>
> --<br>
> 2.5.3<br>
><br>
</span><div><div>> _______________________________________________<br>
> wayland-devel mailing list<br>
> <a href="mailto:wayland-devel@lists.freedesktop.org" target="_blank">wayland-devel@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br>
</div></div></blockquote></div><br></div><div class="gmail_extra">Ok, sorry, I didn't saw that. <div>I will be more careful. </div><div><br></div><div>Thanks</div><div class="gmail_extra"><br clear="all"><div><div dir="ltr"><div>--</div>Lucas Tanure <br><a href="tel:%2B55%20%2819%29%20988176559" value="+5519988176559" target="_blank">+55 (19) 988176559</a></div></div></div></div></div>