net/mlx5: fix Rx queue creation error flow

In Rx queue creation, there are some validations for the Rx configuration.

When one of them fails, the MR btree memory was not freed what caused a memory leak.

Free it.

---
 drivers/net/mlx5/mlx5_rxq.c | 1 +
 1 file changed, 1 insertion(+)

index da7a8b3..dbba5ca 100644
@@ -1689,6 +1689,7 @@ struct mlx5_rxq_ctrl *
 	LIST_INSERT_HEAD(&priv->rxqsctrl, tmpl, next);
 	return tmpl;
 error:
+	mlx5_mr_btree_free(&tmpl->rxq.mr_ctrl.cache_bh);
 	mlx5_free(tmpl);
 	return NULL;
 }

1.8.3.1

In Tx queue creation, there are two validations for the Tx configuration.

When one of them fails, the MR btree memory was not freed what caused a memory leak.

Free it.

---
 drivers/net/mlx5/mlx5_txq.c | 1 +
 1 file changed, 1 insertion(+)

index d96abef..b81bb4a 100644
@@ -1146,6 +1146,7 @@ struct mlx5_txq_ctrl *
 	LIST_INSERT_HEAD(&priv->txqsctrl, tmpl, next);
 	return tmpl;
 error:
+	mlx5_mr_btree_free(&tmpl->txq.mr_ctrl.cache_bh);
 	mlx5_free(tmpl);
 	return NULL;
 }

1.8.3.1

15/12/2020 09:48, Michael Baum:

It is not really fixing the queue creation but a leak. In such case, it is more explicit to say in title:
"fix leak on [...] failure"

Series applied in next-net-mlx with above wording, thanks.