[dpdk-dev] [PATCH] net/e1000: fix out of bounds access

[Lu, Wenzhuo]

Hi Michal,

> -----Original Message-----
> From: Jastrzebski, MichalX K
> Sent: Tuesday, July 25, 2017 9:36 AM
> To: Lu, Wenzhuo <wenzhuo.lu at intel.com>; Kozak, KubaX
> <kubax.kozak at intel.com>
> Cc: dev at dpdk.org; Jain, Deepak K <deepak.k.jain at intel.com>;
> stable at dpdk.org
> Subject: RE: [PATCH] net/e1000: fix out of bounds access
> 
> > -----Original Message-----
> > From: Lu, Wenzhuo
> > Sent: Tuesday, July 25, 2017 3:25 PM
> > To: Kozak, KubaX <kubax.kozak at intel.com>
> > Cc: dev at dpdk.org; Jain, Deepak K <deepak.k.jain at intel.com>;
> > Jastrzebski, MichalX K <michalx.k.jastrzebski at intel.com>;
> > stable at dpdk.org
> > Subject: RE: [PATCH] net/e1000: fix out of bounds access
> >
> > Hi Kuba,
> >
> > > -----Original Message-----
> > > From: Kozak, KubaX
> > > Sent: Tuesday, July 25, 2017 8:19 AM
> > > To: Lu, Wenzhuo <wenzhuo.lu at intel.com>
> > > Cc: dev at dpdk.org; Jain, Deepak K <deepak.k.jain at intel.com>;
> > > Jastrzebski, MichalX K <michalx.k.jastrzebski at intel.com>; Kozak,
> > > KubaX <kubax.kozak at intel.com>; stable at dpdk.org
> > > Subject: [PATCH] net/e1000: fix out of bounds access
> > >
> > > Fix wrong structure type used as argument in memset() call.
> > >
> > > Coverity issue: 147223
> > > Coverity issue: 147227
> > I'm not sure what's this "coverity issue" mean. Maybe we can remove it?
> 
> Hi Wenzhuo,
> these are Coverity issue ID numbers - as this is an issue reported by The
> Coverity Scan tool (a tool for static code analysis).
> So the line " Coverity issue: 147223" means this patch will fix a defect
> reported by The Coverity Scan tool under the number  147223.
Got it. Thanks for that :)

> 
> Best regards
> Michal.
> 
> > Except that, Acked-by: Wenzhuo Lu <wenzhuo.lu at intel.com>
> >
> > > Fixes: a8600af43738 ("net/igb: parse flow API ethertype filter")
> > > Fixes: 22bb13410cb2 ("net/igb: create consistent filter")
> > > Cc: stable at dpdk.org
> > >
> > > Signed-off-by: Kuba Kozak <kubax.kozak at intel.com>