[Date Prev][Date Next]
Re: (ITS#7415) Add MALLOC_CHECK_ and MALLOC_PERTURB_ libc env to the test suite for detecting heap corruption
- To: openldap-its@OpenLDAP.org
- Subject: Re: (ITS#7415) Add MALLOC_CHECK_ and MALLOC_PERTURB_ libc env to the test suite for detecting heap corruption
- From: firstname.lastname@example.org
- Date: Fri, 12 Oct 2012 18:07:09 GMT
- Auto-submitted: auto-generated (OpenLDAP-ITS)
> You miss the fact that we encourage deployers of OpenLDAP Software, who we
recommend build OpenLDAP Software from source, run 'make test' before they
'make install'. We don't want these deployers to have false positives, such as
would be likely caused if we added such environmental variables.
> For those doing automated checks, such as those who do construct packages,
they can have local patches to their hearts content. Likewise for developers.
> So, if it was up to me, I would reject your patch as, IMO, it's in
appropriate for our source distributions. I suspect Howard will chime in
sooner or later.
I think you've already covered the points. I have plenty of checks like this
in my local source tree, but I see no reason to embed OS and platform-specific
stuff into the distributed source. I believe it would lend a false sense of
security, particularly when you're on a platform that ignores these variables,
but you aren't aware of the fact.
It is always the developers' responsibility to know how their own development
environment works, and to tweak it to suit their development efforts.
-- Howard Chu
CTO, Symas Corp. http://www.symas.com
Director, Highland Sun http://highlandsun.com/hyc/
Chief Architect, OpenLDAP http://www.openldap.org/project/