[dpdk-dev] [PATCH v3 1/8] net/failsafe: fix invalid free

[Gaëtan Rivet]

Hi Matan,

On Tue, Jan 09, 2018 at 02:47:26PM +0000, Matan Azrad wrote:
> From: Adrien Mazarguil <adrien.mazarguil at 6wind.com>
> 
> rte_free() is not supposed to work with pointers returned by calloc().
> 
> Fixes: a0194d828100 ("net/failsafe: add flexible device definition")
> Cc: stable at dpdk.org
> 
> Signed-off-by: Adrien Mazarguil <adrien.mazarguil at 6wind.com>
Acked-by: Gaetan Rivet <gaetan.rivet at 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