Don't attempt to compress bss sections
authorAlan Modra <amodra@gmail.com>
Thu, 22 Sep 2022 23:34:52 +0000 (09:04 +0930)
committerAlan Modra <amodra@gmail.com>
Fri, 23 Sep 2022 02:25:47 +0000 (11:55 +0930)
It doesn't make sense to try to compress a section without contents
since those sections take no space on disk.  Compression can only
increase the disk image size.

* coffgen.c (make_a_section_from_file): Exclude !SEC_HAS_CONTENTS
sections from compression and decompression.
* elf.c (_bfd_elf_make_section_from_shdr): Likewise.

bfd/coffgen.c
bfd/elf.c

index 90fba3b855479a03712610771acdf9823379c89d..8eb69af2ce2bc95ac38c6b5f7f3cbdfe9afcd4d1 100644 (file)
@@ -146,7 +146,8 @@ make_a_section_from_file (bfd *abfd,
 
   /* Compress/decompress DWARF debug sections with names: .debug_* and
      .zdebug_*, after the section flags is set.  */
-  if ((flags & SEC_DEBUGGING)
+  if ((flags & SEC_DEBUGGING) != 0
+      && (flags & SEC_HAS_CONTENTS) != 0
       && strlen (name) > 7
       && ((name[1] == 'd' && name[6] == '_')
          || (strlen (name) > 8 && name[1] == 'z' && name[7] == '_')))
index bc433cf516c8d1eabff1e73532c11a752b8e7b85..16cea4f8aeb0e352d4d01181161b66da35c382b6 100644 (file)
--- a/bfd/elf.c
+++ b/bfd/elf.c
@@ -1201,7 +1201,8 @@ _bfd_elf_make_section_from_shdr (bfd *abfd,
 
   /* Compress/decompress DWARF debug sections with names: .debug_* and
      .zdebug_*, after the section flags is set.  */
-  if ((newsect->flags & SEC_DEBUGGING)
+  if ((newsect->flags & SEC_DEBUGGING) != 0
+      && (newsect->flags & SEC_HAS_CONTENTS) != 0
       && ((name[1] == 'd' && name[6] == '_')
          || (name[1] == 'z' && name[7] == '_')))
     {