[PATCH v1] eventdev: fix invalid memory free operation
Jerin Jacob
jerinjacobk at gmail.com
Fri Mar 3 16:55:00 CET 2023
On Wed, Mar 1, 2023 at 3:01 AM Abdullah Sevincer
<abdullah.sevincer at intel.com> wrote:
>
> This commit fixes invalid memory free operation
> caused by extending xstats ID width from 32 to 64 bits.
>
> Telemetry application crashed when freeing memory,
> hence, the telemetry app could not display xstats.
>
> Fixes: 1bdfe4d76e98 ("eventdev: increase xstats ID width to 64 bits")
> Cc: stable at dpdk.org
>
> Signed-off-by: Abdullah Sevincer <abdullah.sevincer at intel.com>
Looks good to me.
Acked-by: Jerin Jacob <jerinj at marvell.com>
Updated the git commit as follows and applied to
dpdk-next-net-eventdev/for-main. Thanks
eventdev: fix invalid memory free operation
Fixe the invalid memory free operation caused by extending
xstats ID width from 32 to 64 bits.
Telemetry application crashed when freeing memory,
hence, the telemetry app could not display xstats.
Fixes: 1bdfe4d76e98 ("eventdev: increase xstats ID width to 64 bits")
Cc: stable at dpdk.org
Signed-off-by: Abdullah Sevincer <abdullah.sevincer at intel.com>
Acked-by: Jerin Jacob <jerinj at marvell.com>
> ---
> lib/eventdev/rte_eventdev.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/eventdev/rte_eventdev.c b/lib/eventdev/rte_eventdev.c
> index 71a29f3506..6ab4524332 100644
> --- a/lib/eventdev/rte_eventdev.c
> +++ b/lib/eventdev/rte_eventdev.c
> @@ -1725,7 +1725,7 @@ eventdev_build_telemetry_data(int dev_id,
> if (xstat_names == NULL)
> return -1;
>
> - ids = malloc((sizeof(unsigned int)) * num_xstats);
> + ids = malloc((sizeof(uint64_t)) * num_xstats);
> if (ids == NULL) {
> free(xstat_names);
> return -1;
> --
> 2.25.1
>
More information about the stable
mailing list