On Tue, Nov 20, 2012 at 5:09 PM, Jiri Kosina <jkosina@xxxxxxx> 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@xxxxxxxxx> > > Signed-off-by: Jiri Kosina <jkosina@xxxxxxx> > --- > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
