This is the mail archive of the
cygwin-developers@cygwin.com
mailing list for the Cygwin project.
Re: Checking input parameters of syscalls
- To: cygwin-developers at cygwin dot com
- Subject: Re: Checking input parameters of syscalls
- From: Christopher Faylor <cgf at redhat dot com>
- Date: Tue, 11 Sep 2001 22:50:54 -0400
- References: <20010910154431.A792@dothill.com> <20010910222228.X937@cygbert.vinschen.de> <1297610035.20010911125836@logos-m.ru>
- Reply-To: cygwin-developers at cygwin dot com
On Tue, Sep 11, 2001 at 12:58:36PM +0400, egor duda wrote:
>Hi!
>
>Tuesday, 11 September, 2001 Corinna Vinschen cygwin-patches@cygwin.com wrote:
>
>CV> Frankly, I don't know. My first guess is to prioritize correctness
>CV> over speed ...
>
>[...]
>
>Speaking about correctness. I'm planning yet another update to
>testsuite and found that many cygwin functions still crash when
>application passes an invalid pointer to them, instead of returning
>EFAULT. Should we leave this as it is (to avoid possible slowdown) or
>sprinkle check_null_empty_str and check_null_invalid_struct all
>around? quick testing shows that IsBadWritePtr() function is
>quite fast (~10 microsecinds per call on my K6/400MHz), so i think
>the latter is the right way to go.
Definitely the latter. There is no reason why cygwin should segv.
Could you add the appropriate checks, Egor?
cgf