[PATCH][weston] ivi_layout_surface_add_notification: Fix potential leak of memory pointed to by 'notification'

Lucas Tanure tanure at linux.com
Tue Sep 22 17:20:46 PDT 2015


Ok. Thanks, I forgot that one.

On Tue, Sep 22, 2015, 21:18 Bryce Harrington <bryce at osg.samsung.com> wrote:

> On Tue, Sep 22, 2015 at 05:18:05PM -0700, Bryce Harrington wrote:
> > On Sat, Sep 19, 2015 at 06:24:58PM -0300, Lucas Tanure wrote:
> > > Signed-off-by: Lucas Tanure <tanure at linux.com>
> >
> > Reviewed-by: Bryce Harrington <bryce at osg.samsung.com>
> >
> > I notice this patch was marked as Superseded in patchwork, but I didn't
> > spot a patch superseding it.  So, I've NEW'd it for application since it
> > still appears to be pertinent.
> >
> > Thanks,
> > Bryce
>
> remote: I: patch #59941 updated using rev
> 193c7a54a82995c87cdded798dc7db33c90763de
> remote: I: 1 patch(es) updated to state Accepted.
> To ssh://git.freedesktop.org/git/wayland/weston
>    96db077..193c7a5  master -> master
>
> > > ---
> > >  ivi-shell/ivi-layout.c | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/ivi-shell/ivi-layout.c b/ivi-shell/ivi-layout.c
> > > index 24bd8dd..061f156 100644
> > > --- a/ivi-shell/ivi-layout.c
> > > +++ b/ivi-shell/ivi-layout.c
> > > @@ -1532,6 +1532,7 @@ ivi_layout_surface_add_notification(struct
> ivi_layout_surface *ivisurf,
> > >     prop_callback = malloc(sizeof *prop_callback);
> > >     if (prop_callback == NULL) {
> > >             weston_log("fails to allocate memory\n");
> > > +           free(notification);
> > >             return IVI_FAILED;
> > >     }
> > >
> > > --
> > > 2.5.3
> > >
> > > _______________________________________________
> > > wayland-devel mailing list
> > > wayland-devel at lists.freedesktop.org
> > > http://lists.freedesktop.org/mailman/listinfo/wayland-devel
> > _______________________________________________
> > wayland-devel mailing list
> > wayland-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20150923/b615a07e/attachment.html>


More information about the wayland-devel mailing list