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
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 > > >

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