Hi Greg,
On 2/9/2024 2:22 AM, Greg KH wrote:
On Thu, Feb 08, 2024 at 03:13:14PM -0800, Wesley Cheng wrote:
From: Mathias Nyman mathias.nyman@linux.intel.com
Don't try to remove a secondary interrupter that is known to be invalid. Also check if the interrupter is valid inside the spinlock that protects the array of interrupters.
Found by smatch static checker
Reported-by: Dan Carpenter dan.carpenter@linaro.org Closes: https://lore.kernel.org/linux-usb/ffaa0a1b-5984-4a1f-bfd3-9184630a97b9@morot... Fixes: c99b38c41234 ("xhci: add support to allocate several interrupters") Signed-off-by: Mathias Nyman mathias.nyman@linux.intel.com Link: https://lore.kernel.org/r/20240125152737.2983959-2-mathias.nyman@linux.intel... Signed-off-by: Greg Kroah-Hartman gregkh@linuxfoundation.org Signed-off-by: Wesley Cheng quic_wcheng@quicinc.com
Wait, this is already in my tree, right? Why keep sending it?
Sorry, I noticed this yesterday night as well when I was preparing some changes to push elsewhere. Will remove the ones I saw that were already present on usb-next.
Thanks Wesley Cheng