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