Re: [PATCH v2] media: mc-device.c: fix memleak in media_device_register_entity

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Sep 06, 2019 at 01:36:02PM +0300, Laurent Pinchart wrote:
> Hi Sakari,
> 
> On Fri, Sep 06, 2019 at 01:11:34PM +0300, Sakari Ailus wrote:
> > On Fri, Sep 06, 2019 at 12:12:03PM +0300, Laurent Pinchart wrote:
> > > On Mon, Aug 19, 2019 at 09:51:30AM +0800, zhengbin wrote:
> > > > In media_device_register_entity, if media_graph_walk_init fails,
> > > > need to free the previously memory.
> > > > 
> > > > Reported-by: Hulk Robot <hulkci@xxxxxxxxxx>
> > > > Signed-off-by: zhengbin <zhengbin13@xxxxxxxxxx>
> > > 
> > > This looks good to me.
> > > 
> > > Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> > > 
> > > and applied to my tree, for v5.5.
> > 
> > Hmm. This is in my tree as well. Would you like to drop it from yours? :-)
> 
> Sure :-)
> 
> I wonder if we should setup a shared git tree for this.

I think following the patchwork status should be enough for now.

I marked it as accepted but forgot to assign it myself in this case. I'll
try assign them as well. It'd be actually nice if Patchwork did that by
default, as this is generally what needs to be done.

What do you think?

If there would be more patches, it'd make sense to rethink this IMO. But
two people eagerly merging patches is much better than none at all. ;)

-- 
Sakari Ailus
sakari.ailus@xxxxxxxxxxxxxxx



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux