Re: [PATCH] firewire: core: fix invalid port index for parent device
Takashi Sakamoto <[email protected]>
| Newsgroups | gmane.linux.kernel.stable,gmane.linux.kernel.firewire.devel,gmane.linux.kernel |
|---|---|
| Message-ID | <[email protected]> |
On Fri, Oct 25, 2024 at 12:41:37PM +0900, Takashi Sakamoto wrote:
> In a commit 24b7f8e5cd65 ("firewire: core: use helper functions for self
> ID sequence"), the enumeration over self ID sequence was refactored with
> some helper functions with KUnit tests. These helper functions are
> guaranteed to work expectedly by the KUnit tests, however their application
> includes a mistake to assign invalid value to the index of port connected
> to parent device.
>
> This bug affects the case that any extra node devices which has three or
> more ports are connected to 1394 OHCI controller. In the case, the path
> to update the tree cache could hits WARN_ON(), and gets general protection
> fault due to the access to invalid address computed by the invalid value.
>
> This commit fixes the bug to assign correct port index.
>
> Cc: [email protected]
> Reported-by: Edmund Raile <[email protected]>
> Closes: https://lore.kernel.org/lkml/[email protected]/
> Fixes: 24b7f8e5cd65 ("firewire: core: use helper functions for self ID sequence")
> Signed-off-by: Takashi Sakamoto <[email protected]>
> ---
> drivers/firewire/core-topology.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied to for-linus branch.
Regards
Takashi Sakamoto