On Thu, Aug 10, 2017 at 06:49:45PM +0300, Sakari Ailus wrote: > From: Rui Miguel Silva <rmfrfs@xxxxxxxxx> > > 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. > > Also use IS_ERR() instead of IS_ERR_OR_NULL() to check return value from > v4l2_flash_init() for it never returns NULL. > > Fixes: 2870b52bae4c ("greybus: lights: add lights implementation") > Reported-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > Signed-off-by: Rui Miguel Silva <rmfrfs@xxxxxxxxx> > Reviewed-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx> > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > Acked-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> > --- > drivers/staging/greybus/light.c | 29 +++++++++-------------------- > 1 file changed, 9 insertions(+), 20 deletions(-) Does not apply to my tree :( _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel