Re: [PATCH] staging: greybus: light: fix a couple double frees

2019-08-29 Thread Rui Miguel Silva
Hi Dan, On Thu 29 Aug 2019 at 13:28, Dan Carpenter wrote: > The problem is in gb_lights_request_handler(). If we get a request to > change the config then we release the light with gb_lights_light_release() > and re-allocated it. However, if the allocation fails part way through > then we call gb

[PATCH] staging: greybus: light: fix a couple double frees

2019-08-29 Thread Dan Carpenter
The problem is in gb_lights_request_handler(). If we get a request to change the config then we release the light with gb_lights_light_release() and re-allocated it. However, if the allocation fails part way through then we call gb_lights_light_release() again. This can lead to a couple differen