Re: [dpdk-dev] [PATCH 1/2] crypto/scheduler: set null pointer after freeing

2018-05-08 Thread De Lara Guarch, Pablo
> -Original Message- > From: Zhang, Roy Fan > Sent: Tuesday, May 8, 2018 12:29 PM > To: De Lara Guarch, Pablo > Cc: dev@dpdk.org; sta...@dpdk.org > Subject: RE: [PATCH 1/2] crypto/scheduler: set null pointer after freeing > > > > > -Original Message- > > From: De Lara Guarch,

Re: [dpdk-dev] [PATCH 1/2] crypto/scheduler: set null pointer after freeing

2018-05-08 Thread Zhang, Roy Fan
> -Original Message- > From: De Lara Guarch, Pablo > Sent: Thursday, April 26, 2018 4:10 PM > To: Zhang, Roy Fan > Cc: dev@dpdk.org; De Lara Guarch, Pablo ; > sta...@dpdk.org > Subject: [PATCH 1/2] crypto/scheduler: set null pointer after freeing > > When freeing memory, pointers should

Re: [dpdk-dev] [PATCH 1/2] crypto/scheduler: set null pointer after freeing

2018-04-27 Thread Bruce Richardson
ta...@dpdk.org > > Subject: Re: [dpdk-dev] [PATCH 1/2] crypto/scheduler: set null pointer after > > freeing > > > > Hi Pablo, > > > > On 4/27/2018 5:06 PM, De Lara Guarch, Pablo wrote: > > > Hi Akhil, > > > > > >> -Original

Re: [dpdk-dev] [PATCH 1/2] crypto/scheduler: set null pointer after freeing

2018-04-27 Thread Van Haaren, Harry
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Akhil Goyal > Sent: Friday, April 27, 2018 12:59 PM > To: De Lara Guarch, Pablo ; Akhil Goyal > ; Zhang, Roy Fan > Cc: dev@dpdk.org; sta...@dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/2] crypto/scheduler: set null point

Re: [dpdk-dev] [PATCH 1/2] crypto/scheduler: set null pointer after freeing

2018-04-27 Thread Akhil Goyal
/2] crypto/scheduler: set null pointer after freeing Hi Pablo, On 4/26/2018 8:39 PM, Pablo de Lara wrote: When freeing memory, pointers should be set to NULL, to avoid memory corruption/segmentation faults. Shouldn't this be handled in the rte_free itself. A lot of other driver are als

Re: [dpdk-dev] [PATCH 1/2] crypto/scheduler: set null pointer after freeing

2018-04-27 Thread De Lara Guarch, Pablo
Hi Akhil, > -Original Message- > From: Akhil Goyal [mailto:akhil.go...@nxp.com] > Sent: Friday, April 27, 2018 9:47 AM > To: De Lara Guarch, Pablo ; Zhang, Roy Fan > > Cc: dev@dpdk.org; sta...@dpdk.org > Subject: Re: [dpdk-dev] [PATCH 1/2] crypto/scheduler: s

Re: [dpdk-dev] [PATCH 1/2] crypto/scheduler: set null pointer after freeing

2018-04-27 Thread Akhil Goyal
Hi Pablo, On 4/26/2018 8:39 PM, Pablo de Lara wrote: When freeing memory, pointers should be set to NULL, to avoid memory corruption/segmentation faults. Shouldn't this be handled in the rte_free itself. A lot of other driver are also not setting null after rte_free. This would require change

[dpdk-dev] [PATCH 1/2] crypto/scheduler: set null pointer after freeing

2018-04-26 Thread Pablo de Lara
When freeing memory, pointers should be set to NULL, to avoid memory corruption/segmentation faults. Fixes: 31439ee72b2c ("crypto/scheduler: add API implementations") Fixes: 50e14527b9d1 ("crypto/scheduler: improve parameters parsing") Fixes: 57523e682bb7 ("crypto/scheduler: register operation fun