Hi Akhil, > -----Original Message----- > From: Akhil Goyal [mailto:[email protected]] > Sent: Friday, April 27, 2018 9:47 AM > To: De Lara Guarch, Pablo <[email protected]>; Zhang, Roy Fan > <[email protected]> > Cc: [email protected]; [email protected] > Subject: Re: [dpdk-dev] [PATCH 1/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 > also not > setting null after rte_free. > This would require change at a lot of places if this is not handled in > rte_free. >
The glibc function "free" works the same way. Users are responsible for setting to NULL these pointers (because sometimes, it is not necessary to do such thing). Anyway, in case we still wanted to change it, we would need to pass a pointer to a pointer in rte_free, which would imply an API breakage. Thanks, Pablo > Thanks, > Akhil
