Re: [PATCH v3 0/2] wifi: mt76: refine USB/SDIO TX path error handling
Devin Wittmayer <[email protected]>
| Newsgroups | org.infradead.lists.linux-mediatek,org.kernel.vger.linux-wireless,org.kernel.vger.stable |
|---|---|
| Message-ID | <[email protected]> |
On Wed, 2026-08-19 at 16:21 +0800, Eason Lai wrote: > This series fixes a UAF in the SDIO TX path when out of memory and > a memory leak in the USB TX path. Thanks for turning that around so fast, and for taking all of it. The code reads right to me now. One thing to sort out before it lands. 2/2 is marked for stable and 1/2 is not, but the two only work together. The padding helper frees the skb itself when it fails, which is fine today because the caller returns without touching it. 2/2 changes that, and starts reporting the prepare failure with a skb that is already gone. 1/2 is what stops the helper freeing. So a stable tree that takes 2/2 on its own gets a use after free where there is no bug today. It will not get caught on the way through, either. 2/2 applies and passes without 1/2, because the frame returns before the padding runs. Marking 1/2 for stable is all it takes. The stable rules say you do not have to list a prerequisite that is already in the same series, but only if that patch is itself marked for stable. The Closes line did not make it onto either patch: https://github.com/morrownr/mt76/issues/83 Devin