From a6efbeceaa32697481a64af7790a07e80c43a667 Mon Sep 17 00:00:00 2001 From: Bruce Korb Date: Tue, 5 Sep 2000 18:29:56 +0000 Subject: [PATCH] always read header files with poorly chosen sizes From-SVN: r36163 --- gcc/ChangeLog | 5 +++++ gcc/fixinc/fixincl.c | 14 ++++++++++++-- 2 files changed, 17 insertions(+), 2 deletions(-) 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 + + * 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 * 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; -- 2.43.5