Fix misaligned access in MachO object file reader: despite containing an
authorRichard Smith <richard-llvm@metafoo.co.uk>
Tue, 21 Aug 2012 20:52:03 +0000 (20:52 +0000)
committerRichard Smith <richard-llvm@metafoo.co.uk>
Tue, 21 Aug 2012 20:52:03 +0000 (20:52 +0000)
int64_t, Symbol64TableEntry is actually only stored with 4-byte alignment
within the file.

The usage of #pragma pack here is copied from the corresponding code in
Support/Endian.h, so shouldn't introduce any new portability problems.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@162312 91177308-0d34-0410-b5e6-96231b3b80d8

include/llvm/Object/MachOFormat.h

index f30d431b69da61449401f5a5fbac0ec2a703c4a7..e4bfcc67fea4d05eb795c0baed0bea91d9c5ae06 100644 (file)
@@ -273,6 +273,10 @@ namespace macho {
     uint16_t Flags;
     uint32_t Value;
   };
+  // Despite containing a uint64_t, this structure is only 4-byte aligned within
+  // a MachO file.
+#pragma pack(push)
+#pragma pack(4)
   struct Symbol64TableEntry {
     uint32_t StringIndex;
     uint8_t Type;
@@ -280,6 +284,7 @@ namespace macho {
     uint16_t Flags;
     uint64_t Value;
   };
+#pragma pack(pop)
 
   /// @}
   /// @name Data-in-code Table Entry