This is the mail archive of the libc-hacker@sources.redhat.com mailing list for the glibc project.

Note that libc-hacker is a closed list. You may look at the archives of this list, but subscription and posting are not open.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Bad alignment in locale-archive


This fixes invalid alignments in the locale-archive file.

Andreas.

2002-10-21  Andreas Schwab  <schwab@suse.de>

	* locale/programs/locarchive.c (create_archive): Properly align
	offsets.
	(enlarge_archive): Likewise.

--- locale/programs/locarchive.c.~1.16.~	2002-10-18 11:14:16.000000000 +0200
+++ locale/programs/locarchive.c	2002-10-21 13:28:27.000000000 +0200
@@ -72,6 +72,9 @@ static const char *locnames[] =
 #define INITIAL_NUM_SUMS	2000
 
 
+#define ALIGN(offset, alignment) \
+  (((offset) + (alignment) - 1) & -(alignment))
+
 static void
 create_archive (const char *archivefname, struct locarhandle *ah)
 {
@@ -90,7 +93,8 @@ create_archive (const char *archivefname
 
   /* Create the initial content of the archive.  */
   head.magic = AR_MAGIC;
-  head.namehash_offset = sizeof (struct locarhead);
+  head.namehash_offset = ALIGN (sizeof (struct locarhead),
+				__alignof__ (struct namehashent));
   head.namehash_used = 0;
   head.namehash_size = next_prime (INITIAL_NUM_NAMES);
 
@@ -99,12 +103,15 @@ create_archive (const char *archivefname
   head.string_used = 0;
   head.string_size = INITIAL_SIZE_STRINGS;
 
-  head.locrectab_offset = head.string_offset + head.string_size;
+  head.locrectab_offset = ALIGN (head.string_offset + head.string_size,
+				 __alignof__ (struct locrecent));
   head.locrectab_used = 0;
   head.locrectab_size = INITIAL_NUM_LOCREC;
 
-  head.sumhash_offset = (head.locrectab_offset
-			 + head.locrectab_size * sizeof (struct locrecent));
+  head.sumhash_offset = ALIGN (head.locrectab_offset
+			       + (head.locrectab_size
+				  * sizeof (struct locrecent)),
+			       __alignof__ (struct sumhashent));
   head.sumhash_used = 0;
   head.sumhash_size = next_prime (INITIAL_NUM_SUMS);
 
@@ -274,13 +281,16 @@ enlarge_archive (struct locarhandle *ah,
 			      * sizeof (struct namehashent)));
   newhead.string_size = MAX (2 * newhead.string_used, newhead.string_size);
 
-  newhead.locrectab_offset = newhead.string_offset + newhead.string_size;
+  newhead.locrectab_offset = ALIGN (newhead.string_offset
+				    + newhead.string_size,
+				    __alignof__ (struct locrecent));
   newhead.locrectab_size = MAX (2 * newhead.locrectab_used,
 				newhead.locrectab_size);
 
-  newhead.sumhash_offset = (newhead.locrectab_offset
-			    + (newhead.locrectab_size
-			       * sizeof (struct locrecent)));
+  newhead.sumhash_offset = ALIGN (newhead.locrectab_offset
+				  + (newhead.locrectab_size
+				     * sizeof (struct locrecent)),
+				  __alignof__ (struct sumhashent));
   newhead.sumhash_size = MAX (next_prime (2 * newhead.sumhash_used),
 			      newhead.sumhash_size);
 

-- 
Andreas Schwab, SuSE Labs, schwab@suse.de
SuSE Linux AG, Deutschherrnstr. 15-19, D-90429 Nürnberg
Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]