Thank you. Can be squashed, but it's OK this way.
Acked-by: Martin Spinler <[email protected]>
On Wed, 2026-02-18 at 10:16 -0800, Stephen Hemminger wrote:
> The eth_node array allocated with calloc() in nfb_nc_eth_init() is
> freed on the error path but not in the normal teardown path via
> nfb_nc_eth_deinit(). Add the missing free().
>
> Fixes: 9b173c0f89 ("net/nfb: get correct link speed")
>
> Signed-off-by: Stephen Hemminger <[email protected]>
> ---
> drivers/net/nfb/nfb_ethdev.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/net/nfb/nfb_ethdev.c b/drivers/net/nfb/nfb_ethdev.c
> index e267dca9af..ba7c849a4b 100644
> --- a/drivers/net/nfb/nfb_ethdev.c
> +++ b/drivers/net/nfb/nfb_ethdev.c
> @@ -186,6 +186,7 @@ nfb_nc_eth_deinit(struct pmd_internals *intl)
> for (i = 0; i < intl->max_rxmac; i++)
> nc_rxmac_close(intl->rxmac[i]);
>
> + free(intl->eth_node);
> free(intl->txmac);
> free(intl->rxmac);
> }
