On Thu, 21 Apr 2022, Benjamin Tissoires wrote: > On Sat, Apr 16, 2022 at 9:37 AM Miaoqian Lin <linmq006@xxxxxxxxx> wrote: > > > > 'input' is a managed resource allocated with devm_input_allocate_device(), > > so there is no need to call input_free_device() explicitly or > > there will be a double free. > > > > According to the doc of devm_input_allocate_device(): > > * Managed input devices do not need to be explicitly unregistered or > > * freed as it will be done automatically when owner device unbinds from > > * its driver (or binding fails). > > > > Fixes: b7429ea53d6c ("HID: elan: Fix memleak in elan_input_configured") > > Fixes: 9a6a4193d65b ("HID: Add driver for USB ELAN Touchpad") > > Signed-off-by: Miaoqian Lin <linmq006@xxxxxxxxx> > > Acked-by: Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx> > > Thanks for the patch! Hmm, this patch never seems to have reached my inbox, but we've had some trouble with our mailserver over the Easter weekend, so that could be it. Thanks for the fix indeed, applied now to hid.git#for-5.18/upstream-fixes -- Jiri Kosina SUSE Labs