Skip to content

Commit 95e65f2

Browse files
isilenceaxboe
authored andcommitted
io_uring/zcrx: fix leaks on failed registration
If we try to register a device-less interface like veth, io_register_zcrx_ifq() will leak struct io_zcrx_ifq with a bunch of resources attached to it. Fix that. Fixes: 035af94 ("io_uring/zcrx: grab a net device") Reported-by: kernel test robot <lkp@intel.com> Reported-by: Dan Carpenter <dan.carpenter@linaro.org> Closes: https://lore.kernel.org/r/202502190532.W7NnmyiP-lkp@intel.com/ Signed-off-by: Pavel Begunkov <asml.silence@gmail.com> Link: https://lore.kernel.org/r/fbf16279dd73fa4c6df048168728355636ba5f53.1739959771.git.asml.silence@gmail.com Signed-off-by: Jens Axboe <axboe@kernel.dk>
1 parent bc674a0 commit 95e65f2

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

io_uring/zcrx.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -388,8 +388,9 @@ int io_register_zcrx_ifq(struct io_ring_ctx *ctx,
388388
goto err;
389389

390390
ifq->dev = ifq->netdev->dev.parent;
391+
ret = -EOPNOTSUPP;
391392
if (!ifq->dev)
392-
return -EOPNOTSUPP;
393+
goto err;
393394
get_device(ifq->dev);
394395

395396
ret = io_zcrx_map_area(ifq, ifq->area);

0 commit comments

Comments
 (0)