[dpdk-dev] [PATCH 0/4] i40e: fix coverity defects

Xing, Beilei beilei.xing at intel.com
Tue Jul 5 03:47:14 CEST 2016


Hi Bruce,

> -----Original Message-----
> From: Richardson, Bruce
> Sent: Monday, July 4, 2016 9:58 PM
> To: Xing, Beilei <beilei.xing at intel.com>
> Cc: Wu, Jingjing <jingjing.wu at intel.com>; Jastrzebski, MichalX K
> <michalx.k.jastrzebski at intel.com>; dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH 0/4] i40e: fix coverity defects
> 
> On Thu, Jun 30, 2016 at 03:34:16PM +0800, Beilei Xing wrote:
> > Fix some open coverity defects.
> >
> > Beilei Xing (4):
> >   i40e: fix wrong operator
> >   i40e: fix dereference before null check
> >   i40e: fix out-of-bounds access
> >   examples/tep_term: fix out-of-bounds access
> >
> >  drivers/net/i40e/i40e_ethdev.c         | 35 ++++++++++++++++++++---------
> -----
> >  drivers/net/i40e/i40e_ethdev_vf.c      |  7 +++++--
> >  examples/tep_termination/vxlan_setup.c |  2 +-
> >  3 files changed, 27 insertions(+), 17 deletions(-)
> >
> > --
> 
> Hi Beilei,
> 
> thanks for the patches. Since these are coverity fixes, can you please do a V2
> with the coverity id's being fixed called out in the commit message. The
> format to use is:
> 
> 	Coverity issue: xxxxx yyyyy ....
> 
> just above the fixes line in the patch. [Use "git log" to see examples in other
> commits].

Thanks for guides:)

> 
> Also, for convenience sake for myself and Thomas, can you include just the
> three i40e patches in one set - since those go to next-net tree - and submit
> the examples fix as a separate standalone patch to go in the main tree.

OK, no problem, will send v2 later.

> 
> Thanks,
> /Bruce



More information about the dev mailing list