[dpdk-dev] [PATCH] lib/telemetry: fix memory leak

Laatz, Kevin kevin.laatz at intel.com
Fri May 31 14:15:56 CEST 2019


On 29/05/2019 13:43, Reshma Pattan wrote:
> Free the `values` pointer before returning
> from rte_telemetry_command_ports_all_stat_values()
> to avoid memory leak.
>
> Fixes: c12cefa379 ("telemetry: fix mapping of statistics")
> CC: stable at dpdk.org
> CC: bruce.richardson at intel.com
>
> Signed-off-by: Reshma Pattan <reshma.pattan at intel.com>
> ---
>   lib/librte_telemetry/rte_telemetry_parser.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/lib/librte_telemetry/rte_telemetry_parser.c b/lib/librte_telemetry/rte_telemetry_parser.c
> index 9bc16eef4..e9297021a 100644
> --- a/lib/librte_telemetry/rte_telemetry_parser.c
> +++ b/lib/librte_telemetry/rte_telemetry_parser.c
> @@ -342,6 +342,7 @@ rte_telemetry_command_ports_all_stat_values(struct telemetry_impl *telemetry,
>   		goto fail;
>   	}
>   
> +	free(values);
>   	return 0;
>   
>   fail:

Acked-by: Kevin Laatz <kevin.laatz at intel.com>



More information about the dev mailing list