Re: [PATCH] HID: roccat: free buffered reports when destroying device
From: Jiri Kosina
Date: Mon Aug 03 2026 - 14:59:49 EST
On Fri, 31 Jul 2026, raoxu wrote:
> From: Xu Rao <raoxu@xxxxxxxxxxxxx>
>
> roccat_report_event() duplicates each report with kmemdup() and stores
> the allocation in a circular-buffer slot. The allocation is released only
> when that slot is reused.
>
> The device destruction paths free struct roccat_device without releasing
> reports still stored in cbuf[]. This makes those allocations unreachable
> and leaks up to ROCCAT_CBUF_SIZE report buffers per device.
>
> Add a small destructor that frees every buffered report before freeing the
> device, and use it in both paths that can destroy a registered device.
>
> Fixes: 206f5f2fcb5f ("HID: roccat: propagate special events of roccat hardware to userspace")
> Cc: stable@xxxxxxxxxxxxxxx
> Signed-off-by: Xu Rao <raoxu@xxxxxxxxxxxxx>
Applied, thanks.
--
Jiri Kosina
SUSE Labs