patches for DPDK stable branches
 help / color / mirror / Atom feed
* [dpdk-stable] [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381
@ 2021-03-17  8:23 Wei Huang
  2021-03-17  8:23 ` [dpdk-stable] [PATCH v1 1/2] raw/ifpga/base: use untained variable as argument Wei Huang
                   ` (2 more replies)
  0 siblings, 3 replies; 12+ messages in thread
From: Wei Huang @ 2021-03-17  8:23 UTC (permalink / raw)
  To: dev, rosen.xu, qi.z.zhang; +Cc: stable, tianfei.zhang, Wei Huang

Below coverity issues are fixed in this patch set.
367480, 367482

Wei Huang (2):
  raw/ifpga/base: use untained variable as argument
  raw/ifpga/base: add argument to use untained variable

 drivers/raw/ifpga/base/ifpga_fme.c | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

-- 
2.29.2


^ permalink raw reply	[flat|nested] 12+ messages in thread

* [dpdk-stable] [PATCH v1 1/2] raw/ifpga/base: use untained variable as argument
  2021-03-17  8:23 [dpdk-stable] [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381 Wei Huang
@ 2021-03-17  8:23 ` Wei Huang
  2021-04-01  7:48   ` Zhang, Tianfei
                     ` (2 more replies)
  2021-03-17  8:23 ` [dpdk-stable] [PATCH v1 2/2] raw/ifpga/base: add argument to use untained variable Wei Huang
  2021-04-01 12:17 ` [dpdk-stable] [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381 Zhang, Qi Z
  2 siblings, 3 replies; 12+ messages in thread
From: Wei Huang @ 2021-03-17  8:23 UTC (permalink / raw)
  To: dev, rosen.xu, qi.z.zhang; +Cc: stable, tianfei.zhang, Wei Huang

In fme_spi_init(), passing tainted expression "fme->max10_dev"
to function "intel_max10_device_remove" has risk. Untainted
variable "max10" should be used.

Coverity issue: 367480
Fixes: 96ebfcf8125c ("raw/ifpga/base: add SPI and MAX10 device driver")

Signed-off-by: Wei Huang <wei.huang@intel.com>
---
 drivers/raw/ifpga/base/ifpga_fme.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/raw/ifpga/base/ifpga_fme.c b/drivers/raw/ifpga/base/ifpga_fme.c
index 43c7b9c3dc..a63f90331f 100644
--- a/drivers/raw/ifpga/base/ifpga_fme.c
+++ b/drivers/raw/ifpga/base/ifpga_fme.c
@@ -1001,7 +1001,7 @@ static int fme_spi_init(struct ifpga_feature *feature)
 	return ret;
 
 max10_fail:
-	intel_max10_device_remove(fme->max10_dev);
+	intel_max10_device_remove(max10);
 spi_fail:
 	altera_spi_release(spi_master);
 	return ret;
-- 
2.29.2


^ permalink raw reply	[flat|nested] 12+ messages in thread

* [dpdk-stable] [PATCH v1 2/2] raw/ifpga/base: add argument to use untained variable
  2021-03-17  8:23 [dpdk-stable] [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381 Wei Huang
  2021-03-17  8:23 ` [dpdk-stable] [PATCH v1 1/2] raw/ifpga/base: use untained variable as argument Wei Huang
@ 2021-03-17  8:23 ` Wei Huang
  2021-04-01  7:48   ` Zhang, Tianfei
  2021-04-01  8:48   ` Xu, Rosen
  2021-04-01 12:17 ` [dpdk-stable] [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381 Zhang, Qi Z
  2 siblings, 2 replies; 12+ messages in thread
From: Wei Huang @ 2021-03-17  8:23 UTC (permalink / raw)
  To: dev, rosen.xu, qi.z.zhang; +Cc: stable, tianfei.zhang, Wei Huang

In fme_nios_spi_init(), passing tainted expression "fme->max10_dev"
to function "fme_get_board_interface" has risk. Add new arguemnt in
function "fme_get_board_interface" to accept untainted variable.

Coverity issue: 367482
Fixes: 96ebfcf8125c ("raw/ifpga/base: add SPI and MAX10 device driver")

Signed-off-by: Wei Huang <wei.huang@intel.com>
---
 drivers/raw/ifpga/base/ifpga_fme.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/drivers/raw/ifpga/base/ifpga_fme.c b/drivers/raw/ifpga/base/ifpga_fme.c
index a63f90331f..4edb9e09e1 100644
--- a/drivers/raw/ifpga/base/ifpga_fme.c
+++ b/drivers/raw/ifpga/base/ifpga_fme.c
@@ -857,7 +857,8 @@ static int board_type_to_info(u32 type,
 	return 0;
 }
 
-static int fme_get_board_interface(struct ifpga_fme_hw *fme)
+static int fme_get_board_interface(struct ifpga_fme_hw *fme,
+	struct intel_max10_device *max10_dev)
 {
 	struct fme_bitstream_id id;
 	struct ifpga_hw *hw;
@@ -911,15 +912,15 @@ static int fme_get_board_interface(struct ifpga_fme_hw *fme)
 			fme->board_info.nums_of_fvl,
 			fme->board_info.ports_per_fvl);
 
-	if (max10_sys_read(fme->max10_dev, FPGA_PAGE_INFO, &val))
+	if (max10_sys_read(max10_dev, FPGA_PAGE_INFO, &val))
 		return -EINVAL;
 	fme->board_info.boot_page = val & 0x7;
 
-	if (max10_sys_read(fme->max10_dev, MAX10_BUILD_VER, &val))
+	if (max10_sys_read(max10_dev, MAX10_BUILD_VER, &val))
 		return -EINVAL;
 	fme->board_info.max10_version = val;
 
-	if (max10_sys_read(fme->max10_dev, NIOS2_FW_VERSION, &val))
+	if (max10_sys_read(max10_dev, NIOS2_FW_VERSION, &val))
 		return -EINVAL;
 	fme->board_info.nios_fw_version = val;
 
@@ -1169,7 +1170,7 @@ static int fme_nios_spi_init(struct ifpga_feature *feature)
 
 	max10->bus = hw->pci_data->bus;
 
-	fme_get_board_interface(fme);
+	fme_get_board_interface(fme, max10);
 
 	mgr->sensor_list = &max10->opae_sensor_list;
 
@@ -1186,7 +1187,7 @@ static int fme_nios_spi_init(struct ifpga_feature *feature)
 	return ret;
 
 spi_fail:
-	intel_max10_device_remove(fme->max10_dev);
+	intel_max10_device_remove(max10);
 release_dev:
 	altera_spi_release(spi_master);
 	return -ENODEV;
-- 
2.29.2


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-stable] [PATCH v1 1/2] raw/ifpga/base: use untained variable as argument
  2021-03-17  8:23 ` [dpdk-stable] [PATCH v1 1/2] raw/ifpga/base: use untained variable as argument Wei Huang
@ 2021-04-01  7:48   ` Zhang, Tianfei
  2021-04-01  8:48   ` Xu, Rosen
  2021-04-07 14:57   ` [dpdk-stable] [dpdk-dev] " Ferruh Yigit
  2 siblings, 0 replies; 12+ messages in thread
From: Zhang, Tianfei @ 2021-04-01  7:48 UTC (permalink / raw)
  To: Huang, Wei, dev, Xu, Rosen, Zhang, Qi Z; +Cc: stable



> -----Original Message-----
> From: Huang, Wei <wei.huang@intel.com>
> Sent: 2021年3月17日 16:24
> To: dev@dpdk.org; Xu, Rosen <rosen.xu@intel.com>; Zhang, Qi Z
> <qi.z.zhang@intel.com>
> Cc: stable@dpdk.org; Zhang, Tianfei <tianfei.zhang@intel.com>; Huang, Wei
> <wei.huang@intel.com>
> Subject: [PATCH v1 1/2] raw/ifpga/base: use untained variable as argument
> 
> In fme_spi_init(), passing tainted expression "fme->max10_dev"
> to function "intel_max10_device_remove" has risk. Untainted variable
> "max10" should be used.
> 
> Coverity issue: 367480
> Fixes: 96ebfcf8125c ("raw/ifpga/base: add SPI and MAX10 device driver")
> 
> Signed-off-by: Wei Huang <wei.huang@intel.com>
> ---
>  drivers/raw/ifpga/base/ifpga_fme.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/raw/ifpga/base/ifpga_fme.c
> b/drivers/raw/ifpga/base/ifpga_fme.c
> index 43c7b9c3dc..a63f90331f 100644
> --- a/drivers/raw/ifpga/base/ifpga_fme.c
> +++ b/drivers/raw/ifpga/base/ifpga_fme.c
> @@ -1001,7 +1001,7 @@ static int fme_spi_init(struct ifpga_feature
> *feature)
>  	return ret;
> 
>  max10_fail:
> -	intel_max10_device_remove(fme->max10_dev);
> +	intel_max10_device_remove(max10);
>  spi_fail:
>  	altera_spi_release(spi_master);
>  	return ret;
> --
Acked-by: Tianfei zhang <Tianfei.zhang@intel.com>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-stable] [PATCH v1 2/2] raw/ifpga/base: add argument to use untained variable
  2021-03-17  8:23 ` [dpdk-stable] [PATCH v1 2/2] raw/ifpga/base: add argument to use untained variable Wei Huang
@ 2021-04-01  7:48   ` Zhang, Tianfei
  2021-04-01  8:48   ` Xu, Rosen
  1 sibling, 0 replies; 12+ messages in thread
From: Zhang, Tianfei @ 2021-04-01  7:48 UTC (permalink / raw)
  To: Huang, Wei, dev, Xu, Rosen, Zhang, Qi Z; +Cc: stable



> -----Original Message-----
> From: Huang, Wei <wei.huang@intel.com>
> Sent: 2021年3月17日 16:24
> To: dev@dpdk.org; Xu, Rosen <rosen.xu@intel.com>; Zhang, Qi Z
> <qi.z.zhang@intel.com>
> Cc: stable@dpdk.org; Zhang, Tianfei <tianfei.zhang@intel.com>; Huang, Wei
> <wei.huang@intel.com>
> Subject: [PATCH v1 2/2] raw/ifpga/base: add argument to use untained
> variable
> 
> In fme_nios_spi_init(), passing tainted expression "fme->max10_dev"
> to function "fme_get_board_interface" has risk. Add new arguemnt in
> function "fme_get_board_interface" to accept untainted variable.
> 
> Coverity issue: 367482
> Fixes: 96ebfcf8125c ("raw/ifpga/base: add SPI and MAX10 device driver")
> 
> Signed-off-by: Wei Huang <wei.huang@intel.com>
> ---
>  drivers/raw/ifpga/base/ifpga_fme.c | 13 +++++++------
>  1 file changed, 7 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/raw/ifpga/base/ifpga_fme.c
> b/drivers/raw/ifpga/base/ifpga_fme.c
> index a63f90331f..4edb9e09e1 100644
> --- a/drivers/raw/ifpga/base/ifpga_fme.c
> +++ b/drivers/raw/ifpga/base/ifpga_fme.c
> @@ -857,7 +857,8 @@ static int board_type_to_info(u32 type,
>  	return 0;
>  }
> 
> -static int fme_get_board_interface(struct ifpga_fme_hw *fme)
> +static int fme_get_board_interface(struct ifpga_fme_hw *fme,
> +	struct intel_max10_device *max10_dev)
>  {
>  	struct fme_bitstream_id id;
>  	struct ifpga_hw *hw;
> @@ -911,15 +912,15 @@ static int fme_get_board_interface(struct
> ifpga_fme_hw *fme)
>  			fme->board_info.nums_of_fvl,
>  			fme->board_info.ports_per_fvl);
> 
> -	if (max10_sys_read(fme->max10_dev, FPGA_PAGE_INFO, &val))
> +	if (max10_sys_read(max10_dev, FPGA_PAGE_INFO, &val))
>  		return -EINVAL;
>  	fme->board_info.boot_page = val & 0x7;
> 
> -	if (max10_sys_read(fme->max10_dev, MAX10_BUILD_VER, &val))
> +	if (max10_sys_read(max10_dev, MAX10_BUILD_VER, &val))
>  		return -EINVAL;
>  	fme->board_info.max10_version = val;
> 
> -	if (max10_sys_read(fme->max10_dev, NIOS2_FW_VERSION, &val))
> +	if (max10_sys_read(max10_dev, NIOS2_FW_VERSION, &val))
>  		return -EINVAL;
>  	fme->board_info.nios_fw_version = val;
> 
> @@ -1169,7 +1170,7 @@ static int fme_nios_spi_init(struct ifpga_feature
> *feature)
> 
>  	max10->bus = hw->pci_data->bus;
> 
> -	fme_get_board_interface(fme);
> +	fme_get_board_interface(fme, max10);
> 
>  	mgr->sensor_list = &max10->opae_sensor_list;
> 
> @@ -1186,7 +1187,7 @@ static int fme_nios_spi_init(struct ifpga_feature
> *feature)
>  	return ret;
> 
>  spi_fail:
> -	intel_max10_device_remove(fme->max10_dev);
> +	intel_max10_device_remove(max10);
>  release_dev:
>  	altera_spi_release(spi_master);
>  	return -ENODEV;
> --
Acked-by: Tianfei zhang <Tianfei.zhang@intel.com>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-stable] [PATCH v1 1/2] raw/ifpga/base: use untained variable as argument
  2021-03-17  8:23 ` [dpdk-stable] [PATCH v1 1/2] raw/ifpga/base: use untained variable as argument Wei Huang
  2021-04-01  7:48   ` Zhang, Tianfei
@ 2021-04-01  8:48   ` Xu, Rosen
  2021-04-07 14:57   ` [dpdk-stable] [dpdk-dev] " Ferruh Yigit
  2 siblings, 0 replies; 12+ messages in thread
From: Xu, Rosen @ 2021-04-01  8:48 UTC (permalink / raw)
  To: Huang, Wei, dev, Zhang,  Qi Z; +Cc: stable, Zhang, Tianfei

Hi,

-----Original Message-----
From: Huang, Wei <wei.huang@intel.com> 
Sent: Wednesday, March 17, 2021 4:24 PM
To: dev@dpdk.org; Xu, Rosen <rosen.xu@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>
Cc: stable@dpdk.org; Zhang, Tianfei <tianfei.zhang@intel.com>; Huang, Wei <wei.huang@intel.com>
Subject: [PATCH v1 1/2] raw/ifpga/base: use untained variable as argument

In fme_spi_init(), passing tainted expression "fme->max10_dev"
to function "intel_max10_device_remove" has risk. Untainted variable "max10" should be used.

Coverity issue: 367480
Fixes: 96ebfcf8125c ("raw/ifpga/base: add SPI and MAX10 device driver")

Signed-off-by: Wei Huang <wei.huang@intel.com>
---
 drivers/raw/ifpga/base/ifpga_fme.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/raw/ifpga/base/ifpga_fme.c b/drivers/raw/ifpga/base/ifpga_fme.c
index 43c7b9c3dc..a63f90331f 100644
--- a/drivers/raw/ifpga/base/ifpga_fme.c
+++ b/drivers/raw/ifpga/base/ifpga_fme.c
@@ -1001,7 +1001,7 @@ static int fme_spi_init(struct ifpga_feature *feature)
 	return ret;
 
 max10_fail:
-	intel_max10_device_remove(fme->max10_dev);
+	intel_max10_device_remove(max10);
 spi_fail:
 	altera_spi_release(spi_master);
 	return ret;
--
2.29.2

Acked-by: Rosen Xu <rosen.xu@intel.com>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-stable] [PATCH v1 2/2] raw/ifpga/base: add argument to use untained variable
  2021-03-17  8:23 ` [dpdk-stable] [PATCH v1 2/2] raw/ifpga/base: add argument to use untained variable Wei Huang
  2021-04-01  7:48   ` Zhang, Tianfei
@ 2021-04-01  8:48   ` Xu, Rosen
  1 sibling, 0 replies; 12+ messages in thread
From: Xu, Rosen @ 2021-04-01  8:48 UTC (permalink / raw)
  To: Huang, Wei, dev, Zhang,  Qi Z; +Cc: stable, Zhang, Tianfei

Hi,

-----Original Message-----
From: Huang, Wei <wei.huang@intel.com> 
Sent: Wednesday, March 17, 2021 4:24 PM
To: dev@dpdk.org; Xu, Rosen <rosen.xu@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>
Cc: stable@dpdk.org; Zhang, Tianfei <tianfei.zhang@intel.com>; Huang, Wei <wei.huang@intel.com>
Subject: [PATCH v1 2/2] raw/ifpga/base: add argument to use untained variable

In fme_nios_spi_init(), passing tainted expression "fme->max10_dev"
to function "fme_get_board_interface" has risk. Add new arguemnt in function "fme_get_board_interface" to accept untainted variable.

Coverity issue: 367482
Fixes: 96ebfcf8125c ("raw/ifpga/base: add SPI and MAX10 device driver")

Signed-off-by: Wei Huang <wei.huang@intel.com>
---
 drivers/raw/ifpga/base/ifpga_fme.c | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/drivers/raw/ifpga/base/ifpga_fme.c b/drivers/raw/ifpga/base/ifpga_fme.c
index a63f90331f..4edb9e09e1 100644
--- a/drivers/raw/ifpga/base/ifpga_fme.c
+++ b/drivers/raw/ifpga/base/ifpga_fme.c
@@ -857,7 +857,8 @@ static int board_type_to_info(u32 type,
 	return 0;
 }
 
-static int fme_get_board_interface(struct ifpga_fme_hw *fme)
+static int fme_get_board_interface(struct ifpga_fme_hw *fme,
+	struct intel_max10_device *max10_dev)
 {
 	struct fme_bitstream_id id;
 	struct ifpga_hw *hw;
@@ -911,15 +912,15 @@ static int fme_get_board_interface(struct ifpga_fme_hw *fme)
 			fme->board_info.nums_of_fvl,
 			fme->board_info.ports_per_fvl);
 
-	if (max10_sys_read(fme->max10_dev, FPGA_PAGE_INFO, &val))
+	if (max10_sys_read(max10_dev, FPGA_PAGE_INFO, &val))
 		return -EINVAL;
 	fme->board_info.boot_page = val & 0x7;
 
-	if (max10_sys_read(fme->max10_dev, MAX10_BUILD_VER, &val))
+	if (max10_sys_read(max10_dev, MAX10_BUILD_VER, &val))
 		return -EINVAL;
 	fme->board_info.max10_version = val;
 
-	if (max10_sys_read(fme->max10_dev, NIOS2_FW_VERSION, &val))
+	if (max10_sys_read(max10_dev, NIOS2_FW_VERSION, &val))
 		return -EINVAL;
 	fme->board_info.nios_fw_version = val;
 
@@ -1169,7 +1170,7 @@ static int fme_nios_spi_init(struct ifpga_feature *feature)
 
 	max10->bus = hw->pci_data->bus;
 
-	fme_get_board_interface(fme);
+	fme_get_board_interface(fme, max10);
 
 	mgr->sensor_list = &max10->opae_sensor_list;
 
@@ -1186,7 +1187,7 @@ static int fme_nios_spi_init(struct ifpga_feature *feature)
 	return ret;
 
 spi_fail:
-	intel_max10_device_remove(fme->max10_dev);
+	intel_max10_device_remove(max10);
 release_dev:
 	altera_spi_release(spi_master);
 	return -ENODEV;
--
2.29.2

Acked-by: Rosen Xu <rosen.xu@intel.com>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-stable] [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381
  2021-03-17  8:23 [dpdk-stable] [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381 Wei Huang
  2021-03-17  8:23 ` [dpdk-stable] [PATCH v1 1/2] raw/ifpga/base: use untained variable as argument Wei Huang
  2021-03-17  8:23 ` [dpdk-stable] [PATCH v1 2/2] raw/ifpga/base: add argument to use untained variable Wei Huang
@ 2021-04-01 12:17 ` Zhang, Qi Z
  2021-04-20  8:09   ` [dpdk-stable] [dpdk-dev] " Ferruh Yigit
  2 siblings, 1 reply; 12+ messages in thread
From: Zhang, Qi Z @ 2021-04-01 12:17 UTC (permalink / raw)
  To: Huang, Wei, dev, Xu, Rosen; +Cc: stable, Zhang, Tianfei



> -----Original Message-----
> From: Huang, Wei <wei.huang@intel.com>
> Sent: Wednesday, March 17, 2021 4:24 PM
> To: dev@dpdk.org; Xu, Rosen <rosen.xu@intel.com>; Zhang, Qi Z
> <qi.z.zhang@intel.com>
> Cc: stable@dpdk.org; Zhang, Tianfei <tianfei.zhang@intel.com>; Huang, Wei
> <wei.huang@intel.com>
> Subject: [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381
> 
> Below coverity issues are fixed in this patch set.
> 367480, 367482
> 
> Wei Huang (2):
>   raw/ifpga/base: use untained variable as argument
>   raw/ifpga/base: add argument to use untained variable
> 
>  drivers/raw/ifpga/base/ifpga_fme.c | 15 ++++++++-------
>  1 file changed, 8 insertions(+), 7 deletions(-)
> 
> --
> 2.29.2

Applied to dpdk-next-net-intel.

Thanks
Qi

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-stable] [dpdk-dev] [PATCH v1 1/2] raw/ifpga/base: use untained variable as argument
  2021-03-17  8:23 ` [dpdk-stable] [PATCH v1 1/2] raw/ifpga/base: use untained variable as argument Wei Huang
  2021-04-01  7:48   ` Zhang, Tianfei
  2021-04-01  8:48   ` Xu, Rosen
@ 2021-04-07 14:57   ` Ferruh Yigit
  2 siblings, 0 replies; 12+ messages in thread
From: Ferruh Yigit @ 2021-04-07 14:57 UTC (permalink / raw)
  To: Wei Huang, dev, rosen.xu, qi.z.zhang; +Cc: stable, tianfei.zhang

On 3/17/2021 8:23 AM, Wei Huang wrote:
> In fme_spi_init(), passing tainted expression "fme->max10_dev"
> to function "intel_max10_device_remove" has risk. Untainted
> variable "max10" should be used.
> 
> Coverity issue: 367480
> Fixes: 96ebfcf8125c ("raw/ifpga/base: add SPI and MAX10 device driver")
> 

Following is from the coverity:
2. tainted_return_value: Function intel_max10_device_probe returns tainted data.
3. var_assign: Assigning: max10 = intel_max10_device_probe(spi_master, 0), which 
taints max10
5. var_assign_var: Assigning: fme->max10_dev = max10. Both are now tainted.


'max10' is tainted at first place, and 'fme->max10_dev' is tainted because of 
'max10'. In this case how replacing 'fme->max10_dev' with 'max10' helps?

Did you verified that change is fixing the coverity issue?


And as the previous one, what is the exact problem, what is tainted data and how 
it is a problem, can you please describe?

> Signed-off-by: Wei Huang <wei.huang@intel.com>
> ---
>   drivers/raw/ifpga/base/ifpga_fme.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/raw/ifpga/base/ifpga_fme.c b/drivers/raw/ifpga/base/ifpga_fme.c
> index 43c7b9c3dc..a63f90331f 100644
> --- a/drivers/raw/ifpga/base/ifpga_fme.c
> +++ b/drivers/raw/ifpga/base/ifpga_fme.c
> @@ -1001,7 +1001,7 @@ static int fme_spi_init(struct ifpga_feature *feature)
>   	return ret;
>   
>   max10_fail:
> -	intel_max10_device_remove(fme->max10_dev);
> +	intel_max10_device_remove(max10);
>   spi_fail:
>   	altera_spi_release(spi_master);
>   	return ret;
> 


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-stable] [dpdk-dev] [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381
  2021-04-01 12:17 ` [dpdk-stable] [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381 Zhang, Qi Z
@ 2021-04-20  8:09   ` Ferruh Yigit
  2021-04-20  9:12     ` Zhang, Qi Z
  0 siblings, 1 reply; 12+ messages in thread
From: Ferruh Yigit @ 2021-04-20  8:09 UTC (permalink / raw)
  To: Zhang, Qi Z, Huang, Wei, dev, Xu, Rosen; +Cc: stable, Zhang, Tianfei

On 4/1/2021 1:17 PM, Zhang, Qi Z wrote:
> 
> 
>> -----Original Message-----
>> From: Huang, Wei <wei.huang@intel.com>
>> Sent: Wednesday, March 17, 2021 4:24 PM
>> To: dev@dpdk.org; Xu, Rosen <rosen.xu@intel.com>; Zhang, Qi Z
>> <qi.z.zhang@intel.com>
>> Cc: stable@dpdk.org; Zhang, Tianfei <tianfei.zhang@intel.com>; Huang, Wei
>> <wei.huang@intel.com>
>> Subject: [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381
>>
>> Below coverity issues are fixed in this patch set.
>> 367480, 367482
>>
>> Wei Huang (2):
>>    raw/ifpga/base: use untained variable as argument
>>    raw/ifpga/base: add argument to use untained variable
>>
>>   drivers/raw/ifpga/base/ifpga_fme.c | 15 ++++++++-------
>>   1 file changed, 8 insertions(+), 7 deletions(-)
>>
>> --
>> 2.29.2
> 
> Applied to dpdk-next-net-intel.
> 

Hi Qi,

What is the status of this set?
It is not merged to your tree, and its patchwork status is 'Rejected'.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-stable] [dpdk-dev] [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381
  2021-04-20  8:09   ` [dpdk-stable] [dpdk-dev] " Ferruh Yigit
@ 2021-04-20  9:12     ` Zhang, Qi Z
  2021-04-20  9:29       ` Zhang, Qi Z
  0 siblings, 1 reply; 12+ messages in thread
From: Zhang, Qi Z @ 2021-04-20  9:12 UTC (permalink / raw)
  To: Yigit, Ferruh, Huang, Wei, dev, Xu, Rosen; +Cc: stable, Zhang, Tianfei



> -----Original Message-----
> From: Yigit, Ferruh <ferruh.yigit@intel.com>
> Sent: Tuesday, April 20, 2021 4:10 PM
> To: Zhang, Qi Z <qi.z.zhang@intel.com>; Huang, Wei <wei.huang@intel.com>;
> dev@dpdk.org; Xu, Rosen <rosen.xu@intel.com>
> Cc: stable@dpdk.org; Zhang, Tianfei <tianfei.zhang@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v1 0/2] Fix coverity issues reported in
> DPDK-26381
> 
> On 4/1/2021 1:17 PM, Zhang, Qi Z wrote:
> >
> >
> >> -----Original Message-----
> >> From: Huang, Wei <wei.huang@intel.com>
> >> Sent: Wednesday, March 17, 2021 4:24 PM
> >> To: dev@dpdk.org; Xu, Rosen <rosen.xu@intel.com>; Zhang, Qi Z
> >> <qi.z.zhang@intel.com>
> >> Cc: stable@dpdk.org; Zhang, Tianfei <tianfei.zhang@intel.com>; Huang,
> >> Wei <wei.huang@intel.com>
> >> Subject: [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381
> >>
> >> Below coverity issues are fixed in this patch set.
> >> 367480, 367482
> >>
> >> Wei Huang (2):
> >>    raw/ifpga/base: use untained variable as argument
> >>    raw/ifpga/base: add argument to use untained variable
> >>
> >>   drivers/raw/ifpga/base/ifpga_fme.c | 15 ++++++++-------
> >>   1 file changed, 8 insertions(+), 7 deletions(-)
> >>
> >> --
> >> 2.29.2
> >
> > Applied to dpdk-next-net-intel.
> >
> 
> Hi Qi,
> 
> What is the status of this set?
> It is not merged to your tree, and its patchwork status is 'Rejected'.

I think the status is wrong updated, and the patches are just dropped after re-sync with next-net, thanks for capture this.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-stable] [dpdk-dev] [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381
  2021-04-20  9:12     ` Zhang, Qi Z
@ 2021-04-20  9:29       ` Zhang, Qi Z
  0 siblings, 0 replies; 12+ messages in thread
From: Zhang, Qi Z @ 2021-04-20  9:29 UTC (permalink / raw)
  To: Yigit, Ferruh, Huang, Wei, dev, Xu, Rosen; +Cc: stable, Zhang, Tianfei



> -----Original Message-----
> From: Zhang, Qi Z
> Sent: Tuesday, April 20, 2021 5:13 PM
> To: Yigit, Ferruh <ferruh.yigit@intel.com>; Huang, Wei
> <wei.huang@intel.com>; dev@dpdk.org; Xu, Rosen <rosen.xu@intel.com>
> Cc: stable@dpdk.org; Zhang, Tianfei <tianfei.zhang@intel.com>
> Subject: RE: [dpdk-dev] [PATCH v1 0/2] Fix coverity issues reported in
> DPDK-26381
> 
> 
> 
> > -----Original Message-----
> > From: Yigit, Ferruh <ferruh.yigit@intel.com>
> > Sent: Tuesday, April 20, 2021 4:10 PM
> > To: Zhang, Qi Z <qi.z.zhang@intel.com>; Huang, Wei
> > <wei.huang@intel.com>; dev@dpdk.org; Xu, Rosen <rosen.xu@intel.com>
> > Cc: stable@dpdk.org; Zhang, Tianfei <tianfei.zhang@intel.com>
> > Subject: Re: [dpdk-dev] [PATCH v1 0/2] Fix coverity issues reported in
> > DPDK-26381
> >
> > On 4/1/2021 1:17 PM, Zhang, Qi Z wrote:
> > >
> > >
> > >> -----Original Message-----
> > >> From: Huang, Wei <wei.huang@intel.com>
> > >> Sent: Wednesday, March 17, 2021 4:24 PM
> > >> To: dev@dpdk.org; Xu, Rosen <rosen.xu@intel.com>; Zhang, Qi Z
> > >> <qi.z.zhang@intel.com>
> > >> Cc: stable@dpdk.org; Zhang, Tianfei <tianfei.zhang@intel.com>;
> > >> Huang, Wei <wei.huang@intel.com>
> > >> Subject: [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381
> > >>
> > >> Below coverity issues are fixed in this patch set.
> > >> 367480, 367482
> > >>
> > >> Wei Huang (2):
> > >>    raw/ifpga/base: use untained variable as argument
> > >>    raw/ifpga/base: add argument to use untained variable
> > >>
> > >>   drivers/raw/ifpga/base/ifpga_fme.c | 15 ++++++++-------
> > >>   1 file changed, 8 insertions(+), 7 deletions(-)
> > >>
> > >> --
> > >> 2.29.2
> > >
> > > Applied to dpdk-next-net-intel.
> > >
> >
> > Hi Qi,
> >
> > What is the status of this set?
> > It is not merged to your tree, and its patchwork status is 'Rejected'.
> 
> I think the status is wrong updated, and the patches are just dropped after
> re-sync with next-net, thanks for capture this.

OK, the patch set should be rejected, as they already be covered by below patch. 
http://patchwork.dpdk.org/project/dpdk/patch/20210408085151.54996-2-wei.huang@intel.com/
So there is no miss operation in path work, just can't remember what happened :)

We are OK now.



^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2021-04-20  9:29 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-17  8:23 [dpdk-stable] [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381 Wei Huang
2021-03-17  8:23 ` [dpdk-stable] [PATCH v1 1/2] raw/ifpga/base: use untained variable as argument Wei Huang
2021-04-01  7:48   ` Zhang, Tianfei
2021-04-01  8:48   ` Xu, Rosen
2021-04-07 14:57   ` [dpdk-stable] [dpdk-dev] " Ferruh Yigit
2021-03-17  8:23 ` [dpdk-stable] [PATCH v1 2/2] raw/ifpga/base: add argument to use untained variable Wei Huang
2021-04-01  7:48   ` Zhang, Tianfei
2021-04-01  8:48   ` Xu, Rosen
2021-04-01 12:17 ` [dpdk-stable] [PATCH v1 0/2] Fix coverity issues reported in DPDK-26381 Zhang, Qi Z
2021-04-20  8:09   ` [dpdk-stable] [dpdk-dev] " Ferruh Yigit
2021-04-20  9:12     ` Zhang, Qi Z
2021-04-20  9:29       ` Zhang, Qi Z

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).