Re: [PATCH] staging: greybus: light: fix memory leak in v4l2 register

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

 



Hi Sakari,
On Wed, Aug 09, 2017 at 12:14:17PM +0300, Sakari Ailus wrote:
> Hi Rui and Johan,
>
> On 08/02/17 19:52, Rui Miguel Silva wrote:
> > We are allocating memory for the v4l2 flash configuration structure and
> > leak it in the normal path. Just use the stack for this as we do not
> > use it outside of this function.
> >
> > Fixes: 2870b52bae4c ("greybus: lights: add lights implementation")
> > Reported-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
> > Signed-off-by: Rui Miguel Silva <rmfrfs@xxxxxxxxx>
>
> My apologies for missing sending the original patch to the appropriate
> recipients the first time.

No problem, this kind of things happens.

>
> Could I take this through the mediatree?

For me that's fine, since you need this for your other patch.
please add the reviewed-by tag from Viresh.

> The second patch in the original set now depends on this one.
> I'll cc you the second patch as well, it will need updates.

Thanks, I will review the changes to greybus light then.

---
Cheers,
	Rui
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux