From e5081f2f8f6f63affdfb386aa72184366bcc50a4 Mon Sep 17 00:00:00 2001 From: Iain Sandoe Date: Wed, 11 Jan 2012 15:24:10 +0000 Subject: [PATCH] bfd: * mach-o.c (bfd_mach_o_build_seg_command): Separate computation of vmsize from filesize. Don't compute offsets or file sizes for zerofill sections. --- bfd/ChangeLog | 6 ++++++ bfd/mach-o.c | 20 ++++++++++++++++---- 2 files changed, 22 insertions(+), 4 deletions(-) diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 21f137c15fe..55ccf7e5d5c 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,9 @@ +2012-01-11 Iain Sandoe + + * mach-o.c (bfd_mach_o_build_seg_command): Separate computation of + vmsize from filesize. Don't compute offsets or file sizes for + zerofill sections. + 2012-01-11 Iain Sandoe * mach-o.c (bfd_mach_o_build_commands): Make the building of each diff --git a/bfd/mach-o.c b/bfd/mach-o.c index 307a8ebd762..6913b1d929d 100644 --- a/bfd/mach-o.c +++ b/bfd/mach-o.c @@ -2023,6 +2023,7 @@ bfd_mach_o_build_seg_command (const char *segment, /* TODO: fix this up for non-MH_OBJECT cases. */ seg->vmaddr = 0; + seg->vmsize = 0; seg->fileoff = mdata->filelen; seg->filesize = 0; @@ -2048,9 +2049,21 @@ bfd_mach_o_build_seg_command (const char *segment, bfd_mach_o_append_section_to_segment (seg, sec); - if (s->size == 0) - s->offset = 0; - else + s->offset = 0; + if (s->size > 0) + { + seg->vmsize = FILE_ALIGN (seg->vmsize, s->align); + seg->vmsize += s->size; + } + + /* Zerofill sections have zero file size & offset, + and are not written. */ + if ((s->flags & BFD_MACH_O_SECTION_TYPE_MASK) == BFD_MACH_O_S_ZEROFILL + || (s->flags & BFD_MACH_O_SECTION_TYPE_MASK) + == BFD_MACH_O_S_GB_ZEROFILL) + continue; + + if (s->size > 0) { mdata->filelen = FILE_ALIGN (mdata->filelen, s->align); s->offset = mdata->filelen; @@ -2062,7 +2075,6 @@ bfd_mach_o_build_seg_command (const char *segment, } seg->filesize = mdata->filelen - seg->fileoff; - seg->vmsize = seg->filesize; return TRUE; } -- 2.30.2