Hi Matan,
On Tue, Jan 09, 2018 at 02:47:26PM +0000, Matan Azrad wrote:
> From: Adrien Mazarguil <adrien.mazarg...@6wind.com>
>
> rte_free() is not supposed to work with pointers returned by calloc().
>
> Fixes: a0194d828100 ("net/failsafe: add flexible device definition")
> Cc: sta...@dpdk.org
>
> Signed-off-by: Adrien Mazarguil <adrien.mazarg...@6wind.com>
Acked-by: Gaetan Rivet <gaetan.ri...@6wind.com>
> ---
> drivers/net/failsafe/failsafe_args.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/failsafe/failsafe_args.c
> b/drivers/net/failsafe/failsafe_args.c
> index cfc83e3..ec63ac9 100644
> --- a/drivers/net/failsafe/failsafe_args.c
> +++ b/drivers/net/failsafe/failsafe_args.c
> @@ -407,7 +407,7 @@ typedef int (parse_cb)(struct rte_eth_dev *dev, const
> char *params,
> uint8_t i;
>
> FOREACH_SUBDEV(sdev, i, dev) {
> - rte_free(sdev->cmdline);
> + free(sdev->cmdline);
> sdev->cmdline = NULL;
> free(sdev->devargs.args);
> sdev->devargs.args = NULL;
> --
> 1.8.3.1
>
--
Gaëtan Rivet
6WIND
