Messages in this thread | | | Date | Tue, 20 Nov 2012 17:34:04 +0100 | Subject | Re: [PATCH] HID: i2c-hid: fix memory leak during probe | From | Benjamin Tissoires <> |
| |
On Tue, Nov 20, 2012 at 5:09 PM, Jiri Kosina <jkosina@suse.cz> wrote: > In case we are returning from i2c_hid_probe() through the 'err' or > err_mem_free labels, there is noone freeing the buffers allocated by > i2c_hid_alloc_buffers().
ouch... thanks Jiri I hope there are not so much others like this one :)
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
> > Signed-off-by: Jiri Kosina <jkosina@suse.cz> > --- > drivers/hid/i2c-hid/i2c-hid.c | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/drivers/hid/i2c-hid/i2c-hid.c b/drivers/hid/i2c-hid/i2c-hid.c > index 11140bd..67ab5b7 100644 > --- a/drivers/hid/i2c-hid/i2c-hid.c > +++ b/drivers/hid/i2c-hid/i2c-hid.c > @@ -892,6 +892,7 @@ err: > if (ihid->irq) > free_irq(ihid->irq, ihid); > > + i2c_hid_free_buffers(ihid); > kfree(ihid); > return ret; > } > > -- > Jiri Kosina > SUSE Labs
| |