asan: Buffer overflow in evax_bfd_print_dst
[binutils-gdb.git] / bfd / section.c
index 3e6ba0c09389c21b149e6fc8dff599050b4f7282..9a1071454f5aa4280c2296424bdf2220a963e7ff 100644 (file)
@@ -1,5 +1,5 @@
 /* Object file "section" support for the BFD library.
-   Copyright (C) 1990-2021 Free Software Foundation, Inc.
+   Copyright (C) 1990-2022 Free Software Foundation, Inc.
    Written by Cygnus Support.
 
    This file is part of BFD, the Binary File Descriptor library.
@@ -551,9 +551,14 @@ CODE_FRAGMENT
 .    struct bfd_section *s;
 .    const char *linked_to_symbol_name;
 .  } map_head, map_tail;
-. {* Points to the output section this section is already assigned to, if any.
-.    This is used when support for non-contiguous memory regions is enabled.  *}
-. struct bfd_section *already_assigned;
+.
+.  {* Points to the output section this section is already assigned to,
+.     if any.  This is used when support for non-contiguous memory
+.     regions is enabled.  *}
+.  struct bfd_section *already_assigned;
+.
+.  {* Explicitly specified section type, if non-zero.  *}
+.  unsigned int type;
 .
 .} asection;
 .
@@ -609,7 +614,7 @@ CODE_FRAGMENT
 .{
 .  return sec->userdata;
 .}
-.static inline bfd_boolean
+.static inline bool
 .bfd_is_com_section (const asection *sec)
 .{
 .  return (sec->flags & SEC_IS_COMMON) != 0;
@@ -619,33 +624,33 @@ CODE_FRAGMENT
 .   because not all callers use the return value.  A macro implementation
 .   would use a comma expression, eg: "((ptr)->foo = val, TRUE)" and some
 .   compilers will complain about comma expressions that have no effect.  *}
-.static inline bfd_boolean
+.static inline bool
 .bfd_set_section_userdata (asection *sec, void *val)
 .{
 .  sec->userdata = val;
-.  return TRUE;
+.  return true;
 .}
 .
-.static inline bfd_boolean
+.static inline bool
 .bfd_set_section_vma (asection *sec, bfd_vma val)
 .{
 .  sec->vma = sec->lma = val;
-.  sec->user_set_vma = TRUE;
-.  return TRUE;
+.  sec->user_set_vma = true;
+.  return true;
 .}
 .
-.static inline bfd_boolean
+.static inline bool
 .bfd_set_section_lma (asection *sec, bfd_vma val)
 .{
 .  sec->lma = val;
-.  return TRUE;
+.  return true;
 .}
 .
-.static inline bfd_boolean
+.static inline bool
 .bfd_set_section_alignment (asection *sec, unsigned int val)
 .{
 .  sec->alignment_power = val;
-.  return TRUE;
+.  return true;
 .}
 .
 .{* These sections are global, and are managed by BFD.  The application
@@ -667,25 +672,25 @@ CODE_FRAGMENT
 .{* Pointer to the indirect section.  *}
 .#define bfd_ind_section_ptr (&_bfd_std_section[3])
 .
-.static inline bfd_boolean
+.static inline bool
 .bfd_is_und_section (const asection *sec)
 .{
 .  return sec == bfd_und_section_ptr;
 .}
 .
-.static inline bfd_boolean
+.static inline bool
 .bfd_is_abs_section (const asection *sec)
 .{
 .  return sec == bfd_abs_section_ptr;
 .}
 .
-.static inline bfd_boolean
+.static inline bool
 .bfd_is_ind_section (const asection *sec)
 .{
 .  return sec == bfd_ind_section_ptr;
 .}
 .
-.static inline bfd_boolean
+.static inline bool
 .bfd_is_const_section (const asection *sec)
 .{
 .  return (sec >= _bfd_std_section
@@ -694,7 +699,7 @@ CODE_FRAGMENT
 .}
 .
 .{* Return TRUE if input section SEC has been discarded.  *}
-.static inline bfd_boolean
+.static inline bool
 .discarded_section (const asection *sec)
 .{
 .  return (!bfd_is_abs_section (sec)
@@ -737,8 +742,8 @@ CODE_FRAGMENT
 .  {* symbol,                    symbol_ptr_ptr,                     *}        \
 .     (struct bfd_symbol *) SYM, &SEC.symbol,                          \
 .                                                                      \
-.  {* map_head, map_tail, already_assigned                           *}        \
-.     { NULL }, { NULL }, NULL                                         \
+.  {* map_head, map_tail, already_assigned, type                     *}        \
+.     { NULL }, { NULL }, NULL,             0                          \
 .                                                                      \
 .    }
 .
@@ -812,12 +817,12 @@ bfd_section_hash_newfunc (struct bfd_hash_entry *entry,
    is useful for things like relocs which are relative to the base
    of a section.  */
 
-bfd_boolean
+bool
 _bfd_generic_new_section_hook (bfd *abfd, asection *newsect)
 {
   newsect->symbol = bfd_make_empty_symbol (abfd);
   if (newsect->symbol == NULL)
-    return FALSE;
+    return false;
 
   newsect->symbol->name = newsect->name;
   newsect->symbol->value = 0;
@@ -825,7 +830,7 @@ _bfd_generic_new_section_hook (bfd *abfd, asection *newsect)
   newsect->symbol->flags = BSF_SECTION_SYM;
 
   newsect->symbol_ptr_ptr = &newsect->symbol;
-  return TRUE;
+  return true;
 }
 
 unsigned int _bfd_section_id = 0x10;  /* id 0 to 3 used by STD_SECTION.  */
@@ -898,7 +903,10 @@ bfd_get_section_by_name (bfd *abfd, const char *name)
 {
   struct section_hash_entry *sh;
 
-  sh = section_hash_lookup (&abfd->section_htab, name, FALSE, FALSE);
+  if (name == NULL)
+    return NULL;
+
+  sh = section_hash_lookup (&abfd->section_htab, name, false, false);
   if (sh != NULL)
     return &sh->section;
 
@@ -982,7 +990,7 @@ SYNOPSIS
        asection *bfd_get_section_by_name_if
          (bfd *abfd,
           const char *name,
-          bfd_boolean (*func) (bfd *abfd, asection *sect, void *obj),
+          bool (*func) (bfd *abfd, asection *sect, void *obj),
           void *obj);
 
 DESCRIPTION
@@ -1000,15 +1008,16 @@ DESCRIPTION
 
 asection *
 bfd_get_section_by_name_if (bfd *abfd, const char *name,
-                           bfd_boolean (*operation) (bfd *,
-                                                     asection *,
-                                                     void *),
+                           bool (*operation) (bfd *, asection *, void *),
                            void *user_storage)
 {
   struct section_hash_entry *sh;
   unsigned long hash;
 
-  sh = section_hash_lookup (&abfd->section_htab, name, FALSE, FALSE);
+  if (name == NULL)
+    return NULL;
+
+  sh = section_hash_lookup (&abfd->section_htab, name, false, false);
   if (sh == NULL)
     return NULL;
 
@@ -1061,7 +1070,7 @@ bfd_get_unique_section_name (bfd *abfd, const char *templat, int *count)
        abort ();
       sprintf (sname + len, ".%d", num++);
     }
-  while (section_hash_lookup (&abfd->section_htab, sname, FALSE, FALSE));
+  while (section_hash_lookup (&abfd->section_htab, sname, false, false));
 
   if (count != NULL)
     *count = num;
@@ -1116,7 +1125,7 @@ bfd_make_section_old_way (bfd *abfd, const char *name)
     {
       struct section_hash_entry *sh;
 
-      sh = section_hash_lookup (&abfd->section_htab, name, TRUE, FALSE);
+      sh = section_hash_lookup (&abfd->section_htab, name, true, false);
       if (sh == NULL)
        return NULL;
 
@@ -1171,7 +1180,7 @@ bfd_make_section_anyway_with_flags (bfd *abfd, const char *name,
       return NULL;
     }
 
-  sh = section_hash_lookup (&abfd->section_htab, name, TRUE, FALSE);
+  sh = section_hash_lookup (&abfd->section_htab, name, true, false);
   if (sh == NULL)
     return NULL;
 
@@ -1256,7 +1265,7 @@ bfd_make_section_with_flags (bfd *abfd, const char *name,
       || strcmp (name, BFD_IND_SECTION_NAME) == 0)
     return NULL;
 
-  sh = section_hash_lookup (&abfd->section_htab, name, TRUE, FALSE);
+  sh = section_hash_lookup (&abfd->section_htab, name, true, false);
   if (sh == NULL)
     return NULL;
 
@@ -1297,7 +1306,7 @@ FUNCTION
        bfd_set_section_flags
 
 SYNOPSIS
-       bfd_boolean bfd_set_section_flags (asection *sec, flagword flags);
+       bool bfd_set_section_flags (asection *sec, flagword flags);
 
 DESCRIPTION
        Set the attributes of the section @var{sec} to the value @var{flags}.
@@ -1311,11 +1320,11 @@ DESCRIPTION
 
 */
 
-bfd_boolean
+bool
 bfd_set_section_flags (asection *section, flagword flags)
 {
   section->flags = flags;
-  return TRUE;
+  return true;
 }
 
 /*
@@ -1389,7 +1398,7 @@ FUNCTION
 SYNOPSIS
        asection *bfd_sections_find_if
          (bfd *abfd,
-          bfd_boolean (*operation) (bfd *abfd, asection *sect, void *obj),
+          bool (*operation) (bfd *abfd, asection *sect, void *obj),
           void *obj);
 
 DESCRIPTION
@@ -1405,7 +1414,7 @@ DESCRIPTION
 
 asection *
 bfd_sections_find_if (bfd *abfd,
-                     bfd_boolean (*operation) (bfd *, asection *, void *),
+                     bool (*operation) (bfd *, asection *, void *),
                      void *user_storage)
 {
   asection *sect;
@@ -1422,7 +1431,7 @@ FUNCTION
        bfd_set_section_size
 
 SYNOPSIS
-       bfd_boolean bfd_set_section_size (asection *sec, bfd_size_type val);
+       bool bfd_set_section_size (asection *sec, bfd_size_type val);
 
 DESCRIPTION
        Set @var{sec} to the size @var{val}. If the operation is
@@ -1434,7 +1443,7 @@ DESCRIPTION
 
 */
 
-bfd_boolean
+bool
 bfd_set_section_size (asection *sec, bfd_size_type val)
 {
   /* Once you've started writing to any section you cannot create or change
@@ -1443,11 +1452,11 @@ bfd_set_section_size (asection *sec, bfd_size_type val)
   if (sec->owner == NULL || sec->owner->output_has_begun)
     {
       bfd_set_error (bfd_error_invalid_operation);
-      return FALSE;
+      return false;
     }
 
   sec->size = val;
-  return TRUE;
+  return true;
 }
 
 /*
@@ -1455,7 +1464,7 @@ FUNCTION
        bfd_set_section_contents
 
 SYNOPSIS
-       bfd_boolean bfd_set_section_contents
+       bool bfd_set_section_contents
          (bfd *abfd, asection *section, const void *data,
           file_ptr offset, bfd_size_type count);
 
@@ -1481,7 +1490,7 @@ DESCRIPTION
 
 */
 
-bfd_boolean
+bool
 bfd_set_section_contents (bfd *abfd,
                          sec_ptr section,
                          const void *location,
@@ -1493,23 +1502,22 @@ bfd_set_section_contents (bfd *abfd,
   if (!(bfd_section_flags (section) & SEC_HAS_CONTENTS))
     {
       bfd_set_error (bfd_error_no_contents);
-      return FALSE;
+      return false;
     }
 
   sz = section->size;
   if ((bfd_size_type) offset > sz
-      || count > sz
-      || offset + count > sz
+      || count > sz - offset
       || count != (size_t) count)
     {
       bfd_set_error (bfd_error_bad_value);
-      return FALSE;
+      return false;
     }
 
   if (!bfd_write_p (abfd))
     {
       bfd_set_error (bfd_error_invalid_operation);
-      return FALSE;
+      return false;
     }
 
   /* Record a copy of the data in memory if desired.  */
@@ -1520,11 +1528,11 @@ bfd_set_section_contents (bfd *abfd,
   if (BFD_SEND (abfd, _bfd_set_section_contents,
                (abfd, section, location, offset, count)))
     {
-      abfd->output_has_begun = TRUE;
-      return TRUE;
+      abfd->output_has_begun = true;
+      return true;
     }
 
-  return FALSE;
+  return false;
 }
 
 /*
@@ -1532,7 +1540,7 @@ FUNCTION
        bfd_get_section_contents
 
 SYNOPSIS
-       bfd_boolean bfd_get_section_contents
+       bool bfd_get_section_contents
          (bfd *abfd, asection *section, void *location, file_ptr offset,
           bfd_size_type count);
 
@@ -1549,7 +1557,7 @@ DESCRIPTION
        <<FALSE>>.
 
 */
-bfd_boolean
+bool
 bfd_get_section_contents (bfd *abfd,
                          sec_ptr section,
                          void *location,
@@ -1561,7 +1569,7 @@ bfd_get_section_contents (bfd *abfd,
   if (section->flags & SEC_CONSTRUCTOR)
     {
       memset (location, 0, (size_t) count);
-      return TRUE;
+      return true;
     }
 
   if (abfd->direction != write_direction && section->rawsize != 0)
@@ -1569,22 +1577,21 @@ bfd_get_section_contents (bfd *abfd,
   else
     sz = section->size;
   if ((bfd_size_type) offset > sz
-      || count > sz
-      || offset + count > sz
+      || count > sz - offset
       || count != (size_t) count)
     {
       bfd_set_error (bfd_error_bad_value);
-      return FALSE;
+      return false;
     }
 
   if (count == 0)
     /* Don't bother.  */
-    return TRUE;
+    return true;
 
   if ((section->flags & SEC_HAS_CONTENTS) == 0)
     {
       memset (location, 0, (size_t) count);
-      return TRUE;
+      return true;
     }
 
   if ((section->flags & SEC_IN_MEMORY) != 0)
@@ -1596,11 +1603,11 @@ bfd_get_section_contents (bfd *abfd,
             error code.  */
          section->flags &= ~ SEC_IN_MEMORY;
          bfd_set_error (bfd_error_invalid_operation);
-         return FALSE;
+         return false;
        }
 
       memmove (location, section->contents + offset, (size_t) count);
-      return TRUE;
+      return true;
     }
 
   return BFD_SEND (abfd, _bfd_get_section_contents,
@@ -1612,7 +1619,7 @@ FUNCTION
        bfd_malloc_and_get_section
 
 SYNOPSIS
-       bfd_boolean bfd_malloc_and_get_section
+       bool bfd_malloc_and_get_section
          (bfd *abfd, asection *section, bfd_byte **buf);
 
 DESCRIPTION
@@ -1620,7 +1627,7 @@ DESCRIPTION
        into a buffer, *@var{buf}, malloc'd by this function.
 */
 
-bfd_boolean
+bool
 bfd_malloc_and_get_section (bfd *abfd, sec_ptr sec, bfd_byte **buf)
 {
   *buf = NULL;
@@ -1631,7 +1638,7 @@ FUNCTION
        bfd_copy_private_section_data
 
 SYNOPSIS
-       bfd_boolean bfd_copy_private_section_data
+       bool bfd_copy_private_section_data
          (bfd *ibfd, asection *isec, bfd *obfd, asection *osec);
 
 DESCRIPTION
@@ -1653,17 +1660,17 @@ FUNCTION
        bfd_generic_is_group_section
 
 SYNOPSIS
-       bfd_boolean bfd_generic_is_group_section (bfd *, const asection *sec);
+       bool bfd_generic_is_group_section (bfd *, const asection *sec);
 
 DESCRIPTION
        Returns TRUE if @var{sec} is a member of a group.
 */
 
-bfd_boolean
+bool
 bfd_generic_is_group_section (bfd *abfd ATTRIBUTE_UNUSED,
                              const asection *sec ATTRIBUTE_UNUSED)
 {
-  return FALSE;
+  return false;
 }
 
 /*
@@ -1689,20 +1696,20 @@ FUNCTION
        bfd_generic_discard_group
 
 SYNOPSIS
-       bfd_boolean bfd_generic_discard_group (bfd *abfd, asection *group);
+       bool bfd_generic_discard_group (bfd *abfd, asection *group);
 
 DESCRIPTION
        Remove all members of @var{group} from the output.
 */
 
-bfd_boolean
+bool
 bfd_generic_discard_group (bfd *abfd ATTRIBUTE_UNUSED,
                           asection *group ATTRIBUTE_UNUSED)
 {
-  return TRUE;
+  return true;
 }
 
-bfd_boolean
+bool
 _bfd_nowrite_set_section_contents (bfd *abfd,
                                   sec_ptr section ATTRIBUTE_UNUSED,
                                   const void *location ATTRIBUTE_UNUSED,