octeon_ep: cancel queued works in probe error path
authorMichal Schmidt <mschmidt@redhat.com>
Thu, 10 Aug 2023 15:01:14 +0000 (17:01 +0200)
committerJakub Kicinski <kuba@kernel.org>
Tue, 15 Aug 2023 02:08:13 +0000 (19:08 -0700)
commit758c91078165ae641b698750a72eafe7968b3756
treea7e9b4c2e8d7c759d3abdf3f49199240f77bc99a
parent607a7a45cdf38c1901e0d81e4e00a2a88786330a
octeon_ep: cancel queued works in probe error path

If it fails to get the devices's MAC address, octep_probe exits while
leaving the delayed work intr_poll_task queued. When the work later
runs, it's a use after free.

Move the cancelation of intr_poll_task from octep_remove into
octep_device_cleanup. This does not change anything in the octep_remove
flow, but octep_device_cleanup is called also in the octep_probe error
path, where the cancelation is needed.

Note that the cancelation of ctrl_mbox_task has to follow
intr_poll_task's, because the ctrl_mbox_task may be queued by
intr_poll_task.

Fixes: 24d4333233b3 ("octeon_ep: poll for control messages")
Signed-off-by: Michal Schmidt <mschmidt@redhat.com>
Link: https://lore.kernel.org/r/20230810150114.107765-5-mschmidt@redhat.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/marvell/octeon_ep/octep_main.c