Re: [dpdk-dev] [PATCH v2] cryptodev: fix NULL pointer dereference

2017-08-01 Thread Thomas Monjalon
01/08/2017 12:48, De Lara Guarch, Pablo: > From: Gonzalez Monroy, Sergio > > On 01/08/2017 10:35, Thomas Monjalon wrote: > > > 01/08/2017 10:13, Sergio Gonzalez Monroy: > > >> On 31/07/2017 20:33, Thomas Monjalon wrote: > > >>> 31/07/2017 11:18, Pablo de Lara: > > When register a crypto driver

Re: [dpdk-dev] [PATCH v2] cryptodev: fix NULL pointer dereference

2017-08-01 Thread De Lara Guarch, Pablo
> -Original Message- > From: Gonzalez Monroy, Sergio > Sent: Tuesday, August 1, 2017 11:18 AM > To: Thomas Monjalon > Cc: De Lara Guarch, Pablo ; > dev@dpdk.org > Subject: Re: [PATCH v2] cryptodev: fix NULL pointer dereference > > On 01/08/2017 10:35, Thomas Monjalon wrote: > > 01/08/20

Re: [dpdk-dev] [PATCH v2] cryptodev: fix NULL pointer dereference

2017-08-01 Thread Sergio Gonzalez Monroy
On 01/08/2017 10:35, Thomas Monjalon wrote: 01/08/2017 10:13, Sergio Gonzalez Monroy: On 31/07/2017 20:33, Thomas Monjalon wrote: 31/07/2017 11:18, Pablo de Lara: When register a crypto driver, a cryptodev driver structure was being allocated, using malloc. Since this call may fail, it is safe

Re: [dpdk-dev] [PATCH v2] cryptodev: fix NULL pointer dereference

2017-08-01 Thread Thomas Monjalon
01/08/2017 10:13, Sergio Gonzalez Monroy: > On 31/07/2017 20:33, Thomas Monjalon wrote: > > 31/07/2017 11:18, Pablo de Lara: > >> When register a crypto driver, a cryptodev driver > >> structure was being allocated, using malloc. > >> Since this call may fail, it is safer to allocate > >> this memo

Re: [dpdk-dev] [PATCH v2] cryptodev: fix NULL pointer dereference

2017-08-01 Thread Sergio Gonzalez Monroy
On 31/07/2017 20:33, Thomas Monjalon wrote: 31/07/2017 11:18, Pablo de Lara: When register a crypto driver, a cryptodev driver structure was being allocated, using malloc. Since this call may fail, it is safer to allocate this memory statically in each PMD, so driver registration will never fail

Re: [dpdk-dev] [PATCH v2] cryptodev: fix NULL pointer dereference

2017-08-01 Thread De Lara Guarch, Pablo
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Monday, July 31, 2017 8:33 PM > To: De Lara Guarch, Pablo > Cc: Gonzalez Monroy, Sergio ; > dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2] cryptodev: fix NUL

Re: [dpdk-dev] [PATCH v2] cryptodev: fix NULL pointer dereference

2017-07-31 Thread Thomas Monjalon
31/07/2017 11:18, Pablo de Lara: > When register a crypto driver, a cryptodev driver > structure was being allocated, using malloc. > Since this call may fail, it is safer to allocate > this memory statically in each PMD, so driver registration > will never fail. > > Coverity issue: 158645 > > Fi