* miscfuncs.cc (check_invalid_virtual_addr): Assure the last page

	in the range is always tested.  Add appropriate const.
	* mmap.cc (mmap_record::aloc_fh): Remove unused static path_conf object.

diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index e0361ea..a6717d0 100644 (file)
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,3 +1,9 @@
+2004-02-26  Brian Ford  <ford@vss.fsi.com>
+
+       * miscfuncs.cc (check_invalid_virtual_addr): Assure the last page
+       in the range is always tested.  Add appropriate const.
+       * mmap.cc (mmap_record::aloc_fh): Remove unused static path_conf object.
+
 2004-02-25  Christopher Faylor  <cgf@redhat.com>
 
        * exceptions.cc (setup_handler): Signal event for any sigwaitinfo if it

diff --git a/winsup/cygwin/miscfuncs.cc b/winsup/cygwin/miscfuncs.cc
index 7947365..d36a824 100644 (file)
--- a/winsup/cygwin/miscfuncs.cc
+++ b/winsup/cygwin/miscfuncs.cc
@@ -218,9 +218,10 @@ int __stdcall
 check_invalid_virtual_addr (const void *s, unsigned sz)
 {
   MEMORY_BASIC_INFORMATION mbuf;
-  void *end;
+  const void *end;
 
-  for (end = (char *) s + sz; s < end; s = (char *) s + mbuf.RegionSize)
+  for (end = (char *) s + sz; s < end;
+       s = (char *) mbuf.BaseAddress + mbuf.RegionSize)
     if (!VirtualQuery (s, &mbuf, sizeof mbuf))
       return EINVAL;
   return 0;

diff --git a/winsup/cygwin/mmap.cc b/winsup/cygwin/mmap.cc
index cacefea..394ed59 100644 (file)
--- a/winsup/cygwin/mmap.cc
+++ b/winsup/cygwin/mmap.cc
@@ -295,7 +295,6 @@ mmap_record::alloc_fh ()
       return &fh_paging_file;
     }
 
-  static path_conv pc; // should be thread safe - CGF
   /* The file descriptor could have been closed or, even
      worse, could have been reused for another file before
      the call to fork(). This requires creating a fhandler