[PATCH] test-skeleton.c: Do not enable M_PERTURB
Florian Weimer
fweimer@redhat.com
Thu Jun 23 15:05:00 GMT 2016
On 06/23/2016 04:49 PM, Siddhesh Poyarekar wrote:
> On Thu, Jun 23, 2016 at 04:40:42PM +0200, Florian Weimer wrote:
>> Over all, this decreases the realism of the tests because
>> it ensures that freshly allocated memory has a well-defined
>> bit pattern. It also causes malloc to take internal paths
>> different from regular application usage, and therefore
>> reduces malloc test coverage.
>
> The well-defined bit pattern is more likely to catch any bad tests
> though, which might make it valuable.
It could also cover up bugs which would otherwise be visible with fresh
allocations which contain only zeros.
Ideally, we'd run all tests twice, with different settings from the
environment (and also with valgrind).
Thanks,
Florian
More information about the Libc-alpha
mailing list