From: Bruce Korb <bkorb@gnu.org>
Date: Tue, 5 Sep 2000 18:29:56 +0000 (+0000)
Subject: always read header files with poorly chosen sizes
X-Git-Tag: prereleases/libstdc++-2.92~4259
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=a6efbeceaa32697481a64af7790a07e80c43a667;p=thirdparty%2Fgcc.git

always read header files with poorly chosen sizes

From-SVN: r36163
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 702fe3e31636..552efaa1598c 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2000-09-05  Bruce Korb  <bkorb@gnu.org>
+
+        * gcc/fixinc/fixincl.c(load_file): always read header files
+        with sizes that are a multiple of the page size.
+
 2000-09-05  Alexandre Oliva  <aoliva@redhat.com>
 
 	* gcse.c (hash_string_1): Add prototype.
diff --git a/gcc/fixinc/fixincl.c b/gcc/fixinc/fixincl.c
index 9736537cbad3..ad5d23ac4324 100644
--- a/gcc/fixinc/fixincl.c
+++ b/gcc/fixinc/fixincl.c
@@ -338,6 +338,10 @@ load_file ( fname )
   if (stbf.st_size == 0)
     return (char*)NULL;
 
+  /*  Make the data map size one larger than the file size for documentation
+      purposes.  Truth is that there will be a following NUL character if
+      the file size is not a multiple of the page size.  If it is a multiple,
+      then this adjustment sometimes fails anyway.  */
   data_map_size = stbf.st_size+1;
   data_map_fd   = open (fname, O_RDONLY);
   ttl_data_size += data_map_size-1;
@@ -352,8 +356,14 @@ load_file ( fname )
 
 #ifdef HAVE_MMAP_FILE
   curr_data_mapped = BOOL_TRUE;
-  res = (char*)mmap ((void*)NULL, data_map_size, PROT_READ, MAP_PRIVATE,
-                     data_map_fd, 0);
+
+  /*  IF the file size is a multiple of the page size,
+      THEN sometimes you will seg fault trying to access a trailing byte */
+  if ((stbf.st_size & (PAGESIZE-1)) == 0)
+    res = (char*)BAD_ADDR;
+  else
+    res = (char*)mmap ((void*)NULL, data_map_size, PROT_READ,
+                       MAP_PRIVATE, data_map_fd, 0);
   if (res == (char*)BAD_ADDR)
     {
       curr_data_mapped = BOOL_FALSE;