> -----Original Message-----
> From: Zhao1, Wei
> Sent: Monday, November 12, 2018 1:25 AM
> To: dev@dpdk.org
> Cc: Zhang, Qi Z <qi.z.zh...@intel.com>; sta...@dpdk.org; Peng, Yuan
> <yuan.p...@intel.com>; Zhao1, Wei <wei.zh...@intel.com>
> Subject: [PATCH] net/i40e: add parameter check for RSS flow init
>
> There need an parameter check for RSS flow init, or it may cause core dump
> if pointer is NULL in memory copy.
>
> Fixes: ac8d22de2394 ("ethdev: flatten RSS configuration in flow API")
>
> Signed-off-by: Wei Zhao <wei.zh...@intel.com>
Acked-by: Qi Zhang <qi.z.zh...@intel.com>
Applied to dpdk-next-net-net.
Thanks
Qi
> ---
> drivers/net/i40e/i40e_ethdev.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c
> index 1c77906..217a8dc 100644
> --- a/drivers/net/i40e/i40e_ethdev.c
> +++ b/drivers/net/i40e/i40e_ethdev.c
> @@ -12552,13 +12552,16 @@ i40e_rss_conf_init(struct
> i40e_rte_flow_rss_conf *out,
> if (in->key_len > RTE_DIM(out->key) ||
> in->queue_num > RTE_DIM(out->queue))
> return -EINVAL;
> + if (!in->key && in->key_len)
> + return -EINVAL;
> + if (out->key && in->key)
> + out->conf.key = memcpy(out->key, in->key, in->key_len);
> out->conf = (struct rte_flow_action_rss){
> .func = in->func,
> .level = in->level,
> .types = in->types,
> .key_len = in->key_len,
> .queue_num = in->queue_num,
> - .key = memcpy(out->key, in->key, in->key_len),
> .queue = memcpy(out->queue, in->queue,
> sizeof(*in->queue) * in->queue_num),
> };
> --
> 2.7.5
