[dsymutil] Fix __LINKEDIT vmsize in dsymutil upgrade path

cachemeifyoucan · cachemeifyoucan · commit a7c680da5182 · 2017-02-01T16:14:52.000-08:00
Summary: dsymutil upgrade path can change the size of segment and it needs to update the vmsize of the segment to reflect the size change. rdar://problem/30290714 Reviewers: friss Subscribers: llvm-commits Differential Revision: https://reviews.llvm.org/D29433 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@293842 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/tools/dsymutil/MachOUtils.cpp b/tools/dsymutil/MachOUtils.cpp
@@ -236,6 +236,8 @@ static void transferSegmentAndSections(
   if (StringRef("__LINKEDIT") == Segment.segname) {
     Segment.fileoff = LinkeditOffset;
     Segment.filesize = LinkeditSize;
+    // Resize vmsize by rounding to the page size.
+    Segment.vmsize = alignTo(LinkeditSize, 0x1000);
   }
 
   // Check if the end address of the last segment and our current

Original file line number	Diff line number	Diff line change
`@@ -236,6 +236,8 @@ static void transferSegmentAndSections(`
`236`	`236`	`if (StringRef("__LINKEDIT") == Segment.segname) {`
`237`	`237`	`Segment.fileoff = LinkeditOffset;`
`238`	`238`	`Segment.filesize = LinkeditSize;`
	`239`	`+ // Resize vmsize by rounding to the page size.`
	`240`	`+ Segment.vmsize = alignTo(LinkeditSize, 0x1000);`
`239`	`241`	`}`
`240`	`242`
`241`	`243`	`// Check if the end address of the last segment and our current`