Re: [dpdk-dev] [v2 PATCH 0/8] Coverity fixes and other cleanups

["Xu, Rosen" <rosen.xu@intel.com>]

Hi David,

Too busy today, sorry for late reply.

> -----Original Message-----
> From: David Marchand [mailto:david.marchand@redhat.com]
> Sent: Friday, November 08, 2019 22:08
> To: Kevin Traynor <ktraynor@redhat.com>
> Cc: dev <dev@dpdk.org>; Xu, Rosen <rosen.xu@intel.com>; Ye, Xiaolong
> <xiaolong.ye@intel.com>
> Subject: Re: [v2 PATCH 0/8] Coverity fixes and other cleanups
> 
> On Wed, Nov 6, 2019 at 8:02 PM Kevin Traynor <ktraynor@redhat.com>
> wrote:
> >
> > Patches 1-4 are Coverity fixes.
> > Patches 5-8 are removing commented out code.
> >
> > v2:
> > - 1/8: removed added NULL check for devargs_all
> > - 2/8: fixed headline to common/cpt
> > - v1 3/9 'net/bnxt: remove logically dead code'
> >   removed as duplicate patch already applied
> >
> > Kevin Traynor (8):
> >   net/pcap: fix argument checks
> >   common/cpt: fix possible NULL deference
> >   net/ipn3ke: fix incorrect commit check logic
> >   net/ipn3ke: remove useless if statement
> >   net/ipn3ke: remove commented out code
> >   compress/octeontx: remove commented out code
> >   event/opdl: remove commented out code
> >   net/bnxt: remove commented out code
> 
> Dropped patch 3 since I did not see a review on it, and I was not feeling
> comfortable with it.
> 
> Applied, thanks.
> 
> 
> --
> David Marchand