Commit e2a2e6c14fe181989187d6b59a79c5fc32961250 | zig

author	Jonathan Marler <johnnymarler@gmail.com> 2021-10-18 15:54:56-06:00
committer	Andrew Kelley <andrew@ziglang.org> 2021-10-21 17:03:44-04:00
log	e2a2e6c14fe181989187d6b59a79c5fc32961250
tree	1eb9ff3e9cadb2f8499e65e779090775607eef1d
parent	da2b615efb14f662bf3aac1f9050174966acd17d

InstallRawStep: handle empty segments

This fixes InstallRawStep to handle the cases when there are empty segments (segments with no sections).  Before this change, if there was an empty segment with no sections, then the fixup of binaryOffsets is skipped.  This fixes that by looping through each segment until a non-empty one is found and then fixing up the sections. This fixed an issue I was having with InstallRawStep for a bootloader I'm writing.

1 files changed, 3 insertions(+), 3 deletions(-)

lib
- std
  - build
    InstallRawStep.zig

lib/std/build/InstallRawStep.zig+3-3

 @@ -96,8 +96,7 @@ const BinaryElfOutput = struct {
         sort.sort(*BinaryElfSegment, self.segments.items, {}, segmentSortCompare);
-        if (self.segments.items.len > 0) {
+        for (self.segments.items) |firstSegment, i| {
-            const firstSegment = self.segments.items[0];
             if (firstSegment.firstSection) |firstSection| {
                 const diff = firstSection.elfOffset - firstSegment.elfOffset;
 @@ -107,9 +106,10 @@ const BinaryElfOutput = struct {
                 const basePhysicalAddress = firstSegment.physicalAddress;
-                for (self.segments.items) |segment| {
+                for (self.segments.items[i + 1 ..]) |segment| {
                     segment.binaryOffset = segment.physicalAddress - basePhysicalAddress;
                 }
+                break;
             }
         }