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

2017-08-09 Thread Rui Miguel Silva
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

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

2017-08-09 Thread Sakari Ailus
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 im

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

2017-08-02 Thread Viresh Kumar
On 02-08-17, 17: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") > R

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

2017-08-02 Thread Rui Miguel Silva
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 Signed-off-by: Rui Miguel Silv