Re: [PATCH] HID: hid-oxp: fix UAF on pending work in remove()

From: Shengzhuo Wei

Date: Mon Aug 03 2026 - 19:55:13 EST


On 2026-08-03 16:33, Dmitry Torokhov wrote:

> What stops these jobs from re-arming? Should it use
> disable_delayed_work_sync() instead?

Agreed — cancel_delayed_work_sync() only drains the running instance; it
doesn't stop the works from being re-queued from the raw_event / sysfs /
LED-brightness paths before the device is fully torn down, so the UAF
window isn't closed. I'll switch all three calls to
disable_delayed_work_sync() in v2.

Do you have any other comments you'd like addressed in v2?

Thanks,
Shengzhuo