diff options
| author | Christopher Faylor <me@cgf.cx> | 2001-06-28 02:19:57 +0000 |
|---|---|---|
| committer | Christopher Faylor <me@cgf.cx> | 2001-06-28 02:19:57 +0000 |
| commit | 7a4078ee340b7f15c839257d6fa895d92abe0224 (patch) | |
| tree | c12bcf5b7e6d1cfcf06276b8055aadc5d779ddd7 /winsup/cygwin/thread.cc | |
| parent | c135faa8b30fd29a8b8e20cda6b07a882a042cbe (diff) | |
| download | cygnal-7a4078ee340b7f15c839257d6fa895d92abe0224.tar.gz cygnal-7a4078ee340b7f15c839257d6fa895d92abe0224.tar.bz2 cygnal-7a4078ee340b7f15c839257d6fa895d92abe0224.zip | |
Change check_null_empty_path* to check_null_empty_str* throughout.
* path.cc (path_conv::check): Add signal protection here since retrieving info
about remote shares can take some time.
* path.h (check_null_empty_str_errno): Convert to a function prototype.
* path.cc (check_null_empty_str): Move to miscfuncs.cc.
* miscfuncs.cc (check_null_empty_str_errno): New function.
(__check_null_invalid_struct): Ditto.
(__check_null_invalid_struct_errno): Ditto.
(check_null_empty_str): Change from VirtualQuery to IsBadWritePtr.
* thread.cc (check_valid_pointer): Ditto.
* resource.cc (getrlimit): Use check_null_invalid_struct macro for checking
validity of pointer.
(setrlimit): Ditto.
Diffstat (limited to 'winsup/cygwin/thread.cc')
| -rw-r--r-- | winsup/cygwin/thread.cc | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/winsup/cygwin/thread.cc b/winsup/cygwin/thread.cc index 26c29c3f2..3c8f16e03 100644 --- a/winsup/cygwin/thread.cc +++ b/winsup/cygwin/thread.cc @@ -739,9 +739,7 @@ verifyable_object::~verifyable_object () int __stdcall check_valid_pointer (void *pointer) { - MEMORY_BASIC_INFORMATION m; - if (!pointer || !VirtualQuery (pointer, &m, sizeof (m)) - || (m.State != MEM_COMMIT)) + if (!pointer || IsBadWritePtr (pointer, sizeof (verifyable_object))) return EFAULT; return 0; } |