Skip to content

net: ethernat: ezchip: bug fixing and code improvments

Pavel Skripkin linux-kernel-mentees lkml netdev 1 version Applied

Status from this message: This series was applied to netdev/net.git (refs/heads/master):

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 0 repliesApplied 3/3

Patches in v1 All 3 applied

#Subject and filesSizeRepliesApplied
1/3net: ethernet: ezchip: fix UAF in nps_enet_removedrivers/net/ethernet/ezchip/nps_enet.c+1 -10 repliesApplied
2/3net: ethernet: ezchip: remove redundant checkdrivers/net/ethernet/ezchip/nps_enet.c+1 -20 repliesApplied
3/3net: ethernet: ezchip: fix error handlingdrivers/net/ethernet/ezchip/nps_enet.c+1 -10 repliesApplied

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/3] net: ethernat: ezchip: bug fixing and code improvments

While manual code reviewing, I found some error in ezchip driver. Two of them looks very dangerous:
1. use-after-free in nps_enet_remove
Accessing netdev private data after free_netdev()

2. wrong error handling of platform_get_irq() It can cause passing negative irq to request_irq()

Also, in 2nd patch I removed redundant check to increase execution speed and make code more straightforward.

Pavel Skripkin (3):
net: ethernet: ezchip: fix UAF in nps_enet_remove
net: ethernet: ezchip: remove redundant check
net: ethernet: ezchip: fix error handling

 drivers/net/ethernet/ezchip/nps_enet.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)
Signature · 6 lines
2.32.0

_______________________________________________
Linux-kernel-mentees mailing list
[email protected]
https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees