Hi David, Too busy today, sorry for late reply.
> -----Original Message----- > From: David Marchand [mailto:[email protected]] > Sent: Friday, November 08, 2019 22:08 > To: Kevin Traynor <[email protected]> > Cc: dev <[email protected]>; Xu, Rosen <[email protected]>; Ye, Xiaolong > <[email protected]> > Subject: Re: [v2 PATCH 0/8] Coverity fixes and other cleanups > > On Wed, Nov 6, 2019 at 8:02 PM Kevin Traynor <[email protected]> > 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

