DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH v1] net/iavf: fix resource leak issue
@ 2022-05-10  0:54 Wenjun Wu
  2022-05-11 11:46 ` Zhang, Qi Z
  0 siblings, 1 reply; 11+ messages in thread
From: Wenjun Wu @ 2022-05-10  0:54 UTC (permalink / raw)
  To: dev, jingjing.wu, beilei.xing, qi.z.zhang; +Cc: Wenjun Wu

This patch fixes resource leak issue reported by coverity.

Coverity issue: 378017
Fixes: b14e8a57b9fe ("net/iavf: support quanta size configuration")

Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
---
 drivers/net/iavf/iavf_ethdev.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/net/iavf/iavf_ethdev.c b/drivers/net/iavf/iavf_ethdev.c
index d1a2b53675..82672841f4 100644
--- a/drivers/net/iavf/iavf_ethdev.c
+++ b/drivers/net/iavf/iavf_ethdev.c
@@ -2188,7 +2188,8 @@ static int iavf_parse_devargs(struct rte_eth_dev *dev)
 	if (ad->devargs.quanta_size < 256 || ad->devargs.quanta_size > 4096 ||
 	    ad->devargs.quanta_size & 0x40) {
 		PMD_INIT_LOG(ERR, "invalid quanta size\n");
-		return -EINVAL;
+		ret = -EINVAL;
+		goto bail;
 	}
 
 bail:
-- 
2.25.1


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

* RE: [PATCH v1] net/iavf: fix resource leak issue
  2022-05-10  0:54 [PATCH v1] net/iavf: fix resource leak issue Wenjun Wu
@ 2022-05-11 11:46 ` Zhang, Qi Z
  2022-05-11 14:52   ` Thomas Monjalon
  0 siblings, 1 reply; 11+ messages in thread
From: Zhang, Qi Z @ 2022-05-11 11:46 UTC (permalink / raw)
  To: Wu, Wenjun1, dev, Wu,  Jingjing, Xing, Beilei



> -----Original Message-----
> From: Wu, Wenjun1 <wenjun1.wu@intel.com>
> Sent: Tuesday, May 10, 2022 8:54 AM
> To: dev@dpdk.org; Wu, Jingjing <jingjing.wu@intel.com>; Xing, Beilei
> <beilei.xing@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>
> Cc: Wu, Wenjun1 <wenjun1.wu@intel.com>
> Subject: [PATCH v1] net/iavf: fix resource leak issue
> 
> This patch fixes resource leak issue reported by coverity.
> 
> Coverity issue: 378017
> Fixes: b14e8a57b9fe ("net/iavf: support quanta size configuration")
> 
> Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>

Acked-by: Qi Zhang <qi.z.zhang@intel.com>

Applied to dpdk-next-net-intel.

Thanks
Qi


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

* Re: [PATCH v1] net/iavf: fix resource leak issue
  2022-05-11 11:46 ` Zhang, Qi Z
@ 2022-05-11 14:52   ` Thomas Monjalon
  2022-05-11 15:31     ` Zhang, Qi Z
  0 siblings, 1 reply; 11+ messages in thread
From: Thomas Monjalon @ 2022-05-11 14:52 UTC (permalink / raw)
  To: Wu, Wenjun1, Zhang, Qi Z; +Cc: dev, Wu, Jingjing, Xing, Beilei

11/05/2022 13:46, Zhang, Qi Z:
> From: Wu, Wenjun1 <wenjun1.wu@intel.com>
> > 
> > This patch fixes resource leak issue reported by coverity.
> > 
> > Coverity issue: 378017
> > Fixes: b14e8a57b9fe ("net/iavf: support quanta size configuration")
> > 
> > Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
> 
> Acked-by: Qi Zhang <qi.z.zhang@intel.com>
> 
> Applied to dpdk-next-net-intel.

In the title, the word "issue" is useless.
It would be more interesting to precise where is the leak.



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

* RE: [PATCH v1] net/iavf: fix resource leak issue
  2022-05-11 14:52   ` Thomas Monjalon
@ 2022-05-11 15:31     ` Zhang, Qi Z
  2022-05-11 21:17       ` Thomas Monjalon
  0 siblings, 1 reply; 11+ messages in thread
From: Zhang, Qi Z @ 2022-05-11 15:31 UTC (permalink / raw)
  To: Thomas Monjalon, Wu, Wenjun1; +Cc: dev, Wu, Jingjing, Xing, Beilei



> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Wednesday, May 11, 2022 10:52 PM
> To: Wu, Wenjun1 <wenjun1.wu@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>
> Cc: dev@dpdk.org; Wu, Jingjing <jingjing.wu@intel.com>; Xing, Beilei
> <beilei.xing@intel.com>
> Subject: Re: [PATCH v1] net/iavf: fix resource leak issue
> 
> 11/05/2022 13:46, Zhang, Qi Z:
> > From: Wu, Wenjun1 <wenjun1.wu@intel.com>
> > >
> > > This patch fixes resource leak issue reported by coverity.
> > >
> > > Coverity issue: 378017
> > > Fixes: b14e8a57b9fe ("net/iavf: support quanta size configuration")
> > >
> > > Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
> >
> > Acked-by: Qi Zhang <qi.z.zhang@intel.com>
> >
> > Applied to dpdk-next-net-intel.
> 
> In the title, the word "issue" is useless.
> It would be more interesting to precise where is the leak.
> 

Added below into commit log in dpdk-next-net-intel.

kvlist should be freed before return an error due to invalid quanta size. 



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

* Re: [PATCH v1] net/iavf: fix resource leak issue
  2022-05-11 15:31     ` Zhang, Qi Z
@ 2022-05-11 21:17       ` Thomas Monjalon
  2022-05-12  0:20         ` Zhang, Qi Z
  0 siblings, 1 reply; 11+ messages in thread
From: Thomas Monjalon @ 2022-05-11 21:17 UTC (permalink / raw)
  To: Wu, Wenjun1, Zhang, Qi Z; +Cc: dev, Wu, Jingjing, Xing, Beilei

11/05/2022 17:31, Zhang, Qi Z:
> From: Thomas Monjalon <thomas@monjalon.net>
> > 11/05/2022 13:46, Zhang, Qi Z:
> > > From: Wu, Wenjun1 <wenjun1.wu@intel.com>
> > > >
> > > > This patch fixes resource leak issue reported by coverity.
> > > >
> > > > Coverity issue: 378017
> > > > Fixes: b14e8a57b9fe ("net/iavf: support quanta size configuration")
> > > >
> > > > Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
> > >
> > > Acked-by: Qi Zhang <qi.z.zhang@intel.com>
> > >
> > > Applied to dpdk-next-net-intel.
> > 
> > In the title, the word "issue" is useless.
> > It would be more interesting to precise where is the leak.
> > 
> 
> Added below into commit log in dpdk-next-net-intel.
> 
> kvlist should be freed before return an error due to invalid quanta size. 

It does not explain what is the scenario. What is the impacted feature?



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

* RE: [PATCH v1] net/iavf: fix resource leak issue
  2022-05-11 21:17       ` Thomas Monjalon
@ 2022-05-12  0:20         ` Zhang, Qi Z
  2022-05-12  7:06           ` Thomas Monjalon
  0 siblings, 1 reply; 11+ messages in thread
From: Zhang, Qi Z @ 2022-05-12  0:20 UTC (permalink / raw)
  To: Thomas Monjalon, Wu, Wenjun1; +Cc: dev, Wu, Jingjing, Xing, Beilei



> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Thursday, May 12, 2022 5:17 AM
> To: Wu, Wenjun1 <wenjun1.wu@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>
> Cc: dev@dpdk.org; Wu, Jingjing <jingjing.wu@intel.com>; Xing, Beilei
> <beilei.xing@intel.com>
> Subject: Re: [PATCH v1] net/iavf: fix resource leak issue
> 
> 11/05/2022 17:31, Zhang, Qi Z:
> > From: Thomas Monjalon <thomas@monjalon.net>
> > > 11/05/2022 13:46, Zhang, Qi Z:
> > > > From: Wu, Wenjun1 <wenjun1.wu@intel.com>
> > > > >
> > > > > This patch fixes resource leak issue reported by coverity.
> > > > >
> > > > > Coverity issue: 378017
> > > > > Fixes: b14e8a57b9fe ("net/iavf: support quanta size configuration")
> > > > >
> > > > > Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
> > > >
> > > > Acked-by: Qi Zhang <qi.z.zhang@intel.com>
> > > >
> > > > Applied to dpdk-next-net-intel.
> > >
> > > In the title, the word "issue" is useless.
> > > It would be more interesting to precise where is the leak.
> > >
> >
> > Added below into commit log in dpdk-next-net-intel.
> >
> > kvlist should be freed before return an error due to invalid quanta size.
> 
> It does not explain what is the scenario. What is the impacted feature?

I think the scenario has been explained as

"kvlist should be freed before return an error due to invalid quanta size"

the impact has been mentioned as "resource leak" I guess this is how Coverity describe the issue

i agree "memory leak" would be better.

> 


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

* Re: [PATCH v1] net/iavf: fix resource leak issue
  2022-05-12  0:20         ` Zhang, Qi Z
@ 2022-05-12  7:06           ` Thomas Monjalon
  2022-05-12  7:21             ` Zhang, Qi Z
  0 siblings, 1 reply; 11+ messages in thread
From: Thomas Monjalon @ 2022-05-12  7:06 UTC (permalink / raw)
  To: Zhang, Qi Z; +Cc: Wu, Wenjun1, dev, Wu, Jingjing, Xing, Beilei

12/05/2022 02:20, Zhang, Qi Z:
> 
> > -----Original Message-----
> > From: Thomas Monjalon <thomas@monjalon.net>
> > Sent: Thursday, May 12, 2022 5:17 AM
> > To: Wu, Wenjun1 <wenjun1.wu@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>
> > Cc: dev@dpdk.org; Wu, Jingjing <jingjing.wu@intel.com>; Xing, Beilei
> > <beilei.xing@intel.com>
> > Subject: Re: [PATCH v1] net/iavf: fix resource leak issue
> > 
> > 11/05/2022 17:31, Zhang, Qi Z:
> > > From: Thomas Monjalon <thomas@monjalon.net>
> > > > 11/05/2022 13:46, Zhang, Qi Z:
> > > > > From: Wu, Wenjun1 <wenjun1.wu@intel.com>
> > > > > >
> > > > > > This patch fixes resource leak issue reported by coverity.
> > > > > >
> > > > > > Coverity issue: 378017
> > > > > > Fixes: b14e8a57b9fe ("net/iavf: support quanta size configuration")
> > > > > >
> > > > > > Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
> > > > >
> > > > > Acked-by: Qi Zhang <qi.z.zhang@intel.com>
> > > > >
> > > > > Applied to dpdk-next-net-intel.
> > > >
> > > > In the title, the word "issue" is useless.
> > > > It would be more interesting to precise where is the leak.
> > > >
> > >
> > > Added below into commit log in dpdk-next-net-intel.
> > >
> > > kvlist should be freed before return an error due to invalid quanta size.
> > 
> > It does not explain what is the scenario. What is the impacted feature?
> 
> I think the scenario has been explained as
> 
> "kvlist should be freed before return an error due to invalid quanta size"
> the impact has been mentioned as "resource leak" I guess this is how Coverity describe the issue

No, it does not say how to reproduce (the scenario).
We have no idea where it is in the code.




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

* RE: [PATCH v1] net/iavf: fix resource leak issue
  2022-05-12  7:06           ` Thomas Monjalon
@ 2022-05-12  7:21             ` Zhang, Qi Z
  2022-05-17  0:59               ` Zhang, Qi Z
  0 siblings, 1 reply; 11+ messages in thread
From: Zhang, Qi Z @ 2022-05-12  7:21 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: Wu, Wenjun1, dev, Wu,  Jingjing, Xing, Beilei



> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Thursday, May 12, 2022 3:07 PM
> To: Zhang, Qi Z <qi.z.zhang@intel.com>
> Cc: Wu, Wenjun1 <wenjun1.wu@intel.com>; dev@dpdk.org; Wu, Jingjing
> <jingjing.wu@intel.com>; Xing, Beilei <beilei.xing@intel.com>
> Subject: Re: [PATCH v1] net/iavf: fix resource leak issue
> 
> 12/05/2022 02:20, Zhang, Qi Z:
> >
> > > -----Original Message-----
> > > From: Thomas Monjalon <thomas@monjalon.net>
> > > Sent: Thursday, May 12, 2022 5:17 AM
> > > To: Wu, Wenjun1 <wenjun1.wu@intel.com>; Zhang, Qi Z
> > > <qi.z.zhang@intel.com>
> > > Cc: dev@dpdk.org; Wu, Jingjing <jingjing.wu@intel.com>; Xing, Beilei
> > > <beilei.xing@intel.com>
> > > Subject: Re: [PATCH v1] net/iavf: fix resource leak issue
> > >
> > > 11/05/2022 17:31, Zhang, Qi Z:
> > > > From: Thomas Monjalon <thomas@monjalon.net>
> > > > > 11/05/2022 13:46, Zhang, Qi Z:
> > > > > > From: Wu, Wenjun1 <wenjun1.wu@intel.com>
> > > > > > >
> > > > > > > This patch fixes resource leak issue reported by coverity.
> > > > > > >
> > > > > > > Coverity issue: 378017
> > > > > > > Fixes: b14e8a57b9fe ("net/iavf: support quanta size
> > > > > > > configuration")
> > > > > > >
> > > > > > > Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
> > > > > >
> > > > > > Acked-by: Qi Zhang <qi.z.zhang@intel.com>
> > > > > >
> > > > > > Applied to dpdk-next-net-intel.
> > > > >
> > > > > In the title, the word "issue" is useless.
> > > > > It would be more interesting to precise where is the leak.
> > > > >
> > > >
> > > > Added below into commit log in dpdk-next-net-intel.
> > > >
> > > > kvlist should be freed before return an error due to invalid quanta size.
> > >
> > > It does not explain what is the scenario. What is the impacted feature?
> >
> > I think the scenario has been explained as
> >
> > "kvlist should be freed before return an error due to invalid quanta size"
> > the impact has been mentioned as "resource leak" I guess this is how
> > Coverity describe the issue
> 
> No, it does not say how to reproduce (the scenario).
> We have no idea where it is in the code.

set an invalid quanta size from devargs will reproduce the issue
> 
> 


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

* RE: [PATCH v1] net/iavf: fix resource leak issue
  2022-05-12  7:21             ` Zhang, Qi Z
@ 2022-05-17  0:59               ` Zhang, Qi Z
  2022-05-17  6:52                 ` Thomas Monjalon
  0 siblings, 1 reply; 11+ messages in thread
From: Zhang, Qi Z @ 2022-05-17  0:59 UTC (permalink / raw)
  To: Zhang, Qi Z, Thomas Monjalon
  Cc: Wu, Wenjun1, dev, Wu,  Jingjing, Xing, Beilei



> -----Original Message-----
> From: Zhang, Qi Z <qi.z.zhang@intel.com>
> Sent: Thursday, May 12, 2022 3:21 PM
> To: Thomas Monjalon <thomas@monjalon.net>
> Cc: Wu, Wenjun1 <wenjun1.wu@intel.com>; dev@dpdk.org; Wu, Jingjing
> <jingjing.wu@intel.com>; Xing, Beilei <beilei.xing@intel.com>
> Subject: RE: [PATCH v1] net/iavf: fix resource leak issue
> 
> 
> 
> > -----Original Message-----
> > From: Thomas Monjalon <thomas@monjalon.net>
> > Sent: Thursday, May 12, 2022 3:07 PM
> > To: Zhang, Qi Z <qi.z.zhang@intel.com>
> > Cc: Wu, Wenjun1 <wenjun1.wu@intel.com>; dev@dpdk.org; Wu, Jingjing
> > <jingjing.wu@intel.com>; Xing, Beilei <beilei.xing@intel.com>
> > Subject: Re: [PATCH v1] net/iavf: fix resource leak issue
> >
> > 12/05/2022 02:20, Zhang, Qi Z:
> > >
> > > > -----Original Message-----
> > > > From: Thomas Monjalon <thomas@monjalon.net>
> > > > Sent: Thursday, May 12, 2022 5:17 AM
> > > > To: Wu, Wenjun1 <wenjun1.wu@intel.com>; Zhang, Qi Z
> > > > <qi.z.zhang@intel.com>
> > > > Cc: dev@dpdk.org; Wu, Jingjing <jingjing.wu@intel.com>; Xing,
> > > > Beilei <beilei.xing@intel.com>
> > > > Subject: Re: [PATCH v1] net/iavf: fix resource leak issue
> > > >
> > > > 11/05/2022 17:31, Zhang, Qi Z:
> > > > > From: Thomas Monjalon <thomas@monjalon.net>
> > > > > > 11/05/2022 13:46, Zhang, Qi Z:
> > > > > > > From: Wu, Wenjun1 <wenjun1.wu@intel.com>
> > > > > > > >
> > > > > > > > This patch fixes resource leak issue reported by coverity.
> > > > > > > >
> > > > > > > > Coverity issue: 378017
> > > > > > > > Fixes: b14e8a57b9fe ("net/iavf: support quanta size
> > > > > > > > configuration")
> > > > > > > >
> > > > > > > > Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
> > > > > > >
> > > > > > > Acked-by: Qi Zhang <qi.z.zhang@intel.com>
> > > > > > >
> > > > > > > Applied to dpdk-next-net-intel.
> > > > > >
> > > > > > In the title, the word "issue" is useless.
> > > > > > It would be more interesting to precise where is the leak.
> > > > > >
> > > > >
> > > > > Added below into commit log in dpdk-next-net-intel.
> > > > >
> > > > > kvlist should be freed before return an error due to invalid quanta size.
> > > >
> > > > It does not explain what is the scenario. What is the impacted feature?
> > >
> > > I think the scenario has been explained as
> > >
> > > "kvlist should be freed before return an error due to invalid quanta size"
> > > the impact has been mentioned as "resource leak" I guess this is how
> > > Coverity describe the issue
> >
> > No, it does not say how to reproduce (the scenario).
> > We have no idea where it is in the code.
> 
> set an invalid quanta size from devargs will reproduce the issue
> >
> >
Refine the Title / commit log in dpdk-next-net-intel as below to answer Thomas's questions

net/iavf: fix memory leak issue

Set an invalid quanta size from devargs will cause memory leak and this
is reported by coverity.

The patch fix the issue by correcting the error handle.


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

* Re: [PATCH v1] net/iavf: fix resource leak issue
  2022-05-17  0:59               ` Zhang, Qi Z
@ 2022-05-17  6:52                 ` Thomas Monjalon
  2022-05-18  4:55                   ` Zhang, Qi Z
  0 siblings, 1 reply; 11+ messages in thread
From: Thomas Monjalon @ 2022-05-17  6:52 UTC (permalink / raw)
  To: Zhang, Qi Z; +Cc: Wu, Wenjun1, dev, Wu, Jingjing, Xing, Beilei

17/05/2022 02:59, Zhang, Qi Z:
> From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > From: Thomas Monjalon <thomas@monjalon.net>
> > > 12/05/2022 02:20, Zhang, Qi Z:
> > > > From: Thomas Monjalon <thomas@monjalon.net>
> > > > > 11/05/2022 17:31, Zhang, Qi Z:
> > > > > > From: Thomas Monjalon <thomas@monjalon.net>
> > > > > > > 11/05/2022 13:46, Zhang, Qi Z:
> > > > > > > > From: Wu, Wenjun1 <wenjun1.wu@intel.com>
> > > > > > > > >
> > > > > > > > > This patch fixes resource leak issue reported by coverity.
> > > > > > > > >
> > > > > > > > > Coverity issue: 378017
> > > > > > > > > Fixes: b14e8a57b9fe ("net/iavf: support quanta size
> > > > > > > > > configuration")
> > > > > > > > >
> > > > > > > > > Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
> > > > > > > >
> > > > > > > > Acked-by: Qi Zhang <qi.z.zhang@intel.com>
> > > > > > > >
> > > > > > > > Applied to dpdk-next-net-intel.
> > > > > > >
> > > > > > > In the title, the word "issue" is useless.
> > > > > > > It would be more interesting to precise where is the leak.
> > > > > > >
> > > > > >
> > > > > > Added below into commit log in dpdk-next-net-intel.
> > > > > >
> > > > > > kvlist should be freed before return an error due to invalid quanta size.
> > > > >
> > > > > It does not explain what is the scenario. What is the impacted feature?
> > > >
> > > > I think the scenario has been explained as
> > > >
> > > > "kvlist should be freed before return an error due to invalid quanta size"
> > > > the impact has been mentioned as "resource leak" I guess this is how
> > > > Coverity describe the issue
> > >
> > > No, it does not say how to reproduce (the scenario).
> > > We have no idea where it is in the code.
> > 
> > set an invalid quanta size from devargs will reproduce the issue
> > >
> > >
> Refine the Title / commit log in dpdk-next-net-intel as below to answer Thomas's questions
> 
> net/iavf: fix memory leak issue

A memory leak is an issue for sure, so no need to add "issue".

> Set an invalid quanta size from devargs will cause memory leak and this

Set -> Setting

> is reported by coverity.
> 
> The patch fix the issue by correcting the error handle.

handle -> handling

Thanks



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

* RE: [PATCH v1] net/iavf: fix resource leak issue
  2022-05-17  6:52                 ` Thomas Monjalon
@ 2022-05-18  4:55                   ` Zhang, Qi Z
  0 siblings, 0 replies; 11+ messages in thread
From: Zhang, Qi Z @ 2022-05-18  4:55 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: Wu, Wenjun1, dev, Wu,  Jingjing, Xing, Beilei



> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Tuesday, May 17, 2022 2:52 PM
> To: Zhang, Qi Z <qi.z.zhang@intel.com>
> Cc: Wu, Wenjun1 <wenjun1.wu@intel.com>; dev@dpdk.org; Wu, Jingjing
> <jingjing.wu@intel.com>; Xing, Beilei <beilei.xing@intel.com>
> Subject: Re: [PATCH v1] net/iavf: fix resource leak issue
> 
> 17/05/2022 02:59, Zhang, Qi Z:
> > From: Zhang, Qi Z <qi.z.zhang@intel.com>
> > > From: Thomas Monjalon <thomas@monjalon.net>
> > > > 12/05/2022 02:20, Zhang, Qi Z:
> > > > > From: Thomas Monjalon <thomas@monjalon.net>
> > > > > > 11/05/2022 17:31, Zhang, Qi Z:
> > > > > > > From: Thomas Monjalon <thomas@monjalon.net>
> > > > > > > > 11/05/2022 13:46, Zhang, Qi Z:
> > > > > > > > > From: Wu, Wenjun1 <wenjun1.wu@intel.com>
> > > > > > > > > >
> > > > > > > > > > This patch fixes resource leak issue reported by coverity.
> > > > > > > > > >
> > > > > > > > > > Coverity issue: 378017
> > > > > > > > > > Fixes: b14e8a57b9fe ("net/iavf: support quanta size
> > > > > > > > > > configuration")
> > > > > > > > > >
> > > > > > > > > > Signed-off-by: Wenjun Wu <wenjun1.wu@intel.com>
> > > > > > > > >
> > > > > > > > > Acked-by: Qi Zhang <qi.z.zhang@intel.com>
> > > > > > > > >
> > > > > > > > > Applied to dpdk-next-net-intel.
> > > > > > > >
> > > > > > > > In the title, the word "issue" is useless.
> > > > > > > > It would be more interesting to precise where is the leak.
> > > > > > > >
> > > > > > >
> > > > > > > Added below into commit log in dpdk-next-net-intel.
> > > > > > >
> > > > > > > kvlist should be freed before return an error due to invalid quanta
> size.
> > > > > >
> > > > > > It does not explain what is the scenario. What is the impacted feature?
> > > > >
> > > > > I think the scenario has been explained as
> > > > >
> > > > > "kvlist should be freed before return an error due to invalid quanta size"
> > > > > the impact has been mentioned as "resource leak" I guess this is
> > > > > how Coverity describe the issue
> > > >
> > > > No, it does not say how to reproduce (the scenario).
> > > > We have no idea where it is in the code.
> > >
> > > set an invalid quanta size from devargs will reproduce the issue
> > > >
> > > >
> > Refine the Title / commit log in dpdk-next-net-intel as below to
> > answer Thomas's questions
> >
> > net/iavf: fix memory leak issue
> 
> A memory leak is an issue for sure, so no need to add "issue".
> 
> > Set an invalid quanta size from devargs will cause memory leak and
> > this
> 
> Set -> Setting
> 
> > is reported by coverity.
> >
> > The patch fix the issue by correcting the error handle.
> 
> handle -> handling
> 
> Thanks
> 

Ok fixed grammar in dpdk-next-net-intel as suggested.


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

end of thread, other threads:[~2022-05-18  4:55 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-10  0:54 [PATCH v1] net/iavf: fix resource leak issue Wenjun Wu
2022-05-11 11:46 ` Zhang, Qi Z
2022-05-11 14:52   ` Thomas Monjalon
2022-05-11 15:31     ` Zhang, Qi Z
2022-05-11 21:17       ` Thomas Monjalon
2022-05-12  0:20         ` Zhang, Qi Z
2022-05-12  7:06           ` Thomas Monjalon
2022-05-12  7:21             ` Zhang, Qi Z
2022-05-17  0:59               ` Zhang, Qi Z
2022-05-17  6:52                 ` Thomas Monjalon
2022-05-18  4:55                   ` 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).