From: Markus Elfring <[email protected]> Date: Wed, 10 Jun 2026 10:02:58 +0200
Use an additional label so that a bit of exception handling can be better reused at the end of an if branch. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <[email protected]> --- drivers/net/ethernet/intel/i40e/i40e_client.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e_client.c b/drivers/net/ethernet/intel/i40e/i40e_client.c index 84a97ca8a6d8..389b8b83afbd 100644 --- a/drivers/net/ethernet/intel/i40e/i40e_client.c +++ b/drivers/net/ethernet/intel/i40e/i40e_client.c @@ -304,15 +304,15 @@ static int i40e_register_auxiliary_dev(struct i40e_info *ldev, const char *name) ldev->aux_dev = aux_dev; ret = ida_alloc(&i40e_client_ida, GFP_KERNEL); - if (ret < 0) { - kfree(i40e_aux_dev); - return ret; - } + if (ret < 0) + goto free_aux_dev; + aux_dev->id = ret; ret = auxiliary_device_init(aux_dev); if (ret < 0) { ida_free(&i40e_client_ida, aux_dev->id); +free_aux_dev: kfree(i40e_aux_dev); return ret; } -- 2.54.0
