[dsymutil] Fix __LINKEDIT vmsize in dsymutil upgrade path

cachemeifyoucan · cachemeifyoucan · commit edd57e8eb03c · 2017-02-02T00:00:13.000Z
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 llvm-svn: 293842
diff --git a/llvm/tools/dsymutil/MachOUtils.cpp b/llvm/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`