Thread (10 messages) 10 messages, 3 authors, 13h ago

Re: [PATCH 3/3] HID: magicmouse: report charge status over Bluetooth

From: Jose Villaseñor Montfort <hidden>
Date: 2026-07-15 06:21:20
Also in: lkml

Hi Alec,

Heads-up on a textual overlap: I posted an independent fix earlier today
that also touches magicmouse_raw_event(), so it will conflict with this
patch:

  HID: magicmouse: prevent unbounded recursion in magicmouse_raw_event()
  https://lore.kernel.org/linux-input/20260715053526.574725-1-pepemontfort@gmail.com/ (local)

It bounds the DOUBLE_REPORT_ID recursion -- a malicious device can chain
0xf7 packets and drive magicmouse_raw_event() deep enough to overflow the
kernel stack. Mechanically it renames the body to
__magicmouse_raw_event(..., bool nested) and adds a small wrapper, which
lands in the same region where your 3/3 inserts
magicmouse_report_charge_status() and its call site. We share the same
base-commit, so whichever goes in first the other needs a trivial rebase.

No action needed on your side; I just wanted it on the radar so it is not
a surprise at apply time. I'm happy to rebase mine on top of your series,
or the maintainers can take them in either order -- the two changes are
independent and straightforward to combine.

Thanks,
Jose
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help