[Swan-dev] Have pfree(P) and pfreeany(P) (P) = NULL

[Antony Antony]

isn't pfree smarter? linked with efence and leak-detective enabled.
Which would be nice in test runs!

/* stomp on memory!   Is another byte value better? */
memset(p, 0xEF, sizeof(union mhdr) + p->i.size);

There was another reason someone, I think Hugh, bring up. 
something like - libreswan code deal with pointers to pointers so assigning 
NULL is not useful. 

may be there is a better reason to assign NULL now!


On Mon, Nov 28, 2016 at 12:07:05PM -0500, Andrew Cagney wrote:
> I'd like the change:
>     pfree(P)
>     pfreeany(P)
> so that "P" is set to NULL after the free.
> 
> While a tacky hack, it might help flush out some reference after
> free's I suspect we've got lurking.
> 
> Andrew
> _______________________________________________
> Swan-dev mailing list
> Swan-dev at lists.libreswan.org
> https://lists.libreswan.org/mailman/listinfo/swan-dev