Re: [PATCH v15 1/1] mctp pcc: Implement MCTP over PCC Transport

[Jakub Kicinski]

On Wed, 15 Jan 2025 14:52:17 -0500 admiyo@xxxxxxxxxxxxxxxxxxxxxx wrote:
> +	rc = devm_add_action_or_reset(dev, mctp_cleanup_netdev, ndev);
> +	if (rc)
> +		goto cleanup_netdev;

devm_add_action_or_reset() calls the cleanup handler on failure,
so just

	return devm_add_action_or_reset(dev, mctp_cleanup_netdev, ndev);

directly. As is you'll double free the netdev on failure.

> +	return rc;
> +cleanup_netdev:
> +	free_netdev(ndev);
-- 
pw-bot: cr