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
_______________________________________________
greybus-dev mailing list
greybus-dev@xxxxxxxxxxxxxxxx
https://lists.linaro.org/mailman/listinfo/greybus-dev
|