From: Ulrich Drepper <drepper@redhat.com>
Date: Fri, 24 Apr 2009 18:47:19 +0000 (+0000)
Subject: * locale/programs/locarchive.c (enlarge_archive): Conserve address
X-Git-Tag: upstream/2.30~13659
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c9edc8891dac165b8446cdd11db4304f9faf01d8;p=external%2Fglibc.git

* locale/programs/locarchive.c (enlarge_archive): Conserve address

	space when temporarily mapping the whole content of the old file.
---

diff --git a/ChangeLog b/ChangeLog
index 2ea0108..35b45d8 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2009-04-24  Ulrich Drepper  <drepper@redhat.com>
 
+	* locale/programs/locarchive.c (enlarge_archive): Conserve address
+	space when temporarily mapping the whole content of the old file.
+
 	[BZ #10100]
 	* misc/hsearch_r.c (hsearch_r): Add back ensurance that hval is
 	not zero.
diff --git a/locale/programs/locarchive.c b/locale/programs/locarchive.c
index 66c5d39..e1e0423 100644
--- a/locale/programs/locarchive.c
+++ b/locale/programs/locarchive.c
@@ -133,14 +133,14 @@ create_archive (const char *archivefname, struct locarhandle *ah)
      address space.  */
   size_t reserved = RESERVE_MMAP_SIZE;
   int xflags = 0;
-  if (total < RESERVE_MMAP_SIZE
+  if (total < reserved
       && ((p = mmap64 (NULL, reserved, PROT_NONE, MAP_ANON, -1, 0))
 	  != MAP_FAILED))
     xflags = MAP_FIXED;
   else
     {
       p = NULL;
-      reserved = 0;
+      reserved = total;
     }
 
   /* Map the header and all the administration data structures.  */
@@ -307,11 +307,22 @@ enlarge_archive (struct locarhandle *ah, const struct locarhead *head)
 
   /* Not all of the old file has to be mapped.  Change this now this
      we will have to access the whole content.  */
-  if (fstat64 (ah->fd, &st) != 0
-      || (st.st_size > ah->mmaped
-	  && (ah->addr = mmap64 (NULL, st.st_size, PROT_READ | PROT_WRITE,
-				 MAP_SHARED, ah->fd, 0)) == MAP_FAILED))
+  if (fstat64 (ah->fd, &st) != 0)
+  enomap:
     error (EXIT_FAILURE, errno, _("cannot map locale archive file"));
+
+  if (st.st_size < ah->reserved)
+    ah->addr = mremap (ah->addr, ah->mmaped, st.st_size,
+		       MREMAP_MAYMOVE | MREMAP_FIXED, ah->addr);
+  else
+    {
+      munmap (ah->addr, ah->reserved);
+      ah->addr = mmap64 (NULL, st.st_size, PROT_READ | PROT_WRITE,
+			 MAP_SHARED, ah->fd, 0);
+      ah->reserved = st.st_size;
+    }
+  if (ah->addr == MAP_FAILED)
+    goto enomap;
   ah->mmaped = st.st_size;
 
   /* Create a temporary file in the correct directory.  */
@@ -581,14 +592,14 @@ open_archive (struct locarhandle *ah, bool readonly)
   size_t reserved = RESERVE_MMAP_SIZE;
   int xflags = 0;
   void *p;
-  if (st.st_size < RESERVE_MMAP_SIZE
+  if (st.st_size < reserved
       && ((p = mmap64 (NULL, RESERVE_MMAP_SIZE, PROT_NONE, MAP_ANON, -1, 0))
 	  != MAP_FAILED))
     xflags = MAP_FIXED;
   else
     {
       p = NULL;
-      reserved = 0;
+      reserved = st.st_size;
     }
 
   /* Map the entire file.  We might need to compare the category data
@@ -609,7 +620,7 @@ close_archive (struct locarhandle *ah)
 {
   if (ah->fd != -1)
     {
-      munmap (ah->addr, MAX (ah->reserved, ah->mmaped));
+      munmap (ah->addr, ah->reserved);
       close (ah->fd);
     }
 }