Re: CVS commit: src/bin/ps

[Roy Marples <roy%marples.name@localhost>]

On 11/09/2019 20:03, Robert Elz wrote:
>      Date:        Wed, 11 Sep 2019 17:02:53 +0000
>      From:        "Kamil Rytarowski" <kamil%netbsd.org@localhost>
>      Message-ID:  <20190911170253.D097FFBF4%cvs.NetBSD.org@localhost>
>
>    | Free it when no longer used, just before the program termination.
>
> Can we please avoid this kind of nonsense.   Everything is freed when
> every program exits - doing explicit free() calls makes the program bigger
> and slower for no reason at all.
>
> By all means fix places where memory is truly leaked (whenre more is
> continuously allocated, and simply discarded) but anything that is supposed
> to remain until program exit should simply go away when the exit happens.
>
> If the canitisers cannot be instructed to ignore such things, they are
> much less useful tan they could be.

Could we add an #ifdef for this?

#ifdef __SANITISATION
free(foo);
close(bar);
#endif

return EXIT_SUCCESS;