> -----Original Message-----
> From: Yigit, Ferruh <ferruh.yi...@intel.com>
> Sent: Thursday, April 22, 2021 12:21 AM
> To: Igor Russkikh <igor.russk...@aquantia.com>; Pavel Belous
> <pavel.bel...@aquantia.com>; Somalapuram Amaranath
> <asoma...@amd.com>; Ajit Khaparde <ajit.khapa...@broadcom.com>;
> Somnath Kotur <somnath.ko...@broadcom.com>; Hemant Agrawal
> <hemant.agra...@nxp.com>; Sachin Saxena <sachin.sax...@oss.nxp.com>;
> Guo, Jia <jia....@intel.com>; Wang, Haiyue <haiyue.w...@intel.com>;
> Daley, John <johnd...@cisco.com>; Hyong Youb Kim <hyon...@cisco.com>;
> Min Hu (Connor) <humi...@huawei.com>; Yisen Zhuang
> <yisen.zhu...@huawei.com>; Lijun Ou <ouli...@huawei.com>; Xing, Beilei
> <beilei.x...@intel.com>; Yang, Qiming <qiming.y...@intel.com>; Zhang, Qi Z
> <qi.z.zh...@intel.com>; Andrew Boyer <abo...@pensando.io>; Jerin Jacob
> <jer...@marvell.com>; Nithin Dabilpuram <ndabilpu...@marvell.com>;
> Kiran Kumar K <kirankum...@marvell.com>; Rasesh Mody
> <rm...@marvell.com>; Devendra Singh Rawat <dsinghra...@marvell.com>;
> Andrew Rybchenko <andrew.rybche...@oktetlabs.ru>; Jiawen Wu
> <jiawe...@trustnetic.com>; Jian Wang <jianw...@trustnetic.com>;
> Thomas Monjalon <tho...@monjalon.net>; Selwin Sebastian
> <selwin.sebast...@amd.com>; Remy Horton <remy.hor...@intel.com>;
> Chunsong Feng <fengchuns...@huawei.com>; Huisong Li
> <lihuis...@huawei.com>; Hao Chen <chenhao...@huawei.com>; Wei Hu
> (Xavier) <xavier.hu...@huawei.com>; Wu, Jingjing <jingjing...@intel.com>;
> Lu, Wenzhuo <wenzhuo...@intel.com>; Li, Xiaoyun <xiaoyun...@intel.com>;
> Zhang, AlvinX <alvinx.zh...@intel.com>; Shannon Nelson
> <snel...@pensando.io>; Alfredo Cardigliano <cardigli...@ntop.org>;
> Vamsi Attunuru <vattun...@marvell.com>; Yash Sharma
> <ysha...@marvell.com>; Ivan Malov <ivan.ma...@oktetlabs.ru>; Andrew
> Lee <a...@solarflare.com>
> Cc: Yigit, Ferruh <ferruh.yi...@intel.com>; dev@dpdk.org; sta...@dpdk.org
> Subject: [PATCH] drivers/net: fix FW version get
>
> Fixes a few different things:
> * Remove 'fw_version' NULL checks, it is allowed if the 'fw_size' is
> zero, 'fw_version' being NULL but 'fw_size' not zero condition checked
> in ethdev layer
> * Be sure required buffer size is returned if provided one is not big
> enough, instead of returning success (0)
> * Document in doxygen comment the '-EINVAL' is a valid return type
> * Take into account that 'snprintf' can return negative value
> * Cast length to 'size_t' to compare it with 'fw_size'
>
> Fixes: bb42aa9ffe4e ("net/atlantic: configure device start/stop")
> Fixes: ff70acdf4299 ("net/axgbe: support reading FW version")
> Fixes: e2652b0a20a0 ("net/bnxt: support get FW version")
> Fixes: cf0fab1d2ca5 ("net/dpaa: support firmware version get API")
> Fixes: 748eccb97cdc ("net/dpaa2: add support for firmware version get")
> Fixes: b883c0644a24 ("net/e1000: add firmware version get")
> Fixes: 293430677e9c ("net/enic: add handler to return firmware version")
> Fixes: 1f5ca0b460cd ("net/hns3: support some device operations")
> Fixes: bd5b86732bc7 ("net/hns3: modify format for firmware version")
> Fixes: ed0dfdd0e976 ("net/i40e: add firmware version get")
> Fixes: e31cb9a36298 ("net/ice: support FW version getting")
> Fixes: 4f09bc55ac3d ("net/igc: implement device base operations")
> Fixes: eec10fb0ce6b ("net/ionic: support FW version")
> Fixes: 8b0b56574269 ("net/ixgbe: add firmware version get")
> Fixes: 4d9f5b8adc02 ("net/octeontx2: add FW version get operation")
> Fixes: f97b56f9f12e ("net/qede: support FW version query")
> Fixes: 83fef46a22b2 ("net/sfc: add callback to retrieve FW version")
> Fixes: bc84ac0fadef ("net/txgbe: support getting FW version")
> Fixes: 21913471202f ("ethdev: add firmware version get")
> Cc: sta...@dpdk.org
>
> Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com>
> ---
<...>
> diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c
> index e6206a7e5100..66d23d698ea0 100644
> --- a/drivers/net/i40e/i40e_ethdev.c
> +++ b/drivers/net/i40e/i40e_ethdev.c
> @@ -3687,9 +3687,11 @@ i40e_fw_version_get(struct rte_eth_dev *dev,
> char *fw_version, size_t fw_size)
> ((hw->nvm.version >> 4) & 0xff),
> (hw->nvm.version & 0xf), hw->nvm.eetrack,
> ver, build, patch);
> + if (ret < 0)
> + return -EINVAL;
>
> ret += 1; /* add the size of '\0' */
> - if (fw_size < (u32)ret)
> + if (fw_size < (size_t)ret)
> return ret;
> else
> return 0;
For i40e,
Acked-by: Beilei Xing <beilei.x...@intel.com>
