Try harder to find the correct symbol to associate with OPEN GNU BUILD notes.
[binutils-gdb.git] / binutils / readelf.c
index 5cccccb24f1fdcc281053d006533f8ec70a36994..34781ae86636c84186847aa4e1163554e4d6c7e6 100644 (file)
@@ -2205,11 +2205,11 @@ get_machine_name (unsigned e_machine)
 
   switch (e_machine)
     {
+      /* Please keep this switch table sorted by increasing EM_ value.  */
+      /* 0 */
     case EM_NONE:              return _("None");
-    case EM_AARCH64:           return "AArch64";
     case EM_M32:               return "WE32100";
     case EM_SPARC:             return "Sparc";
-    case EM_SPU:               return "SPU";
     case EM_386:               return "Intel 80386";
     case EM_68K:               return "MC68000";
     case EM_88K:               return "MC88000";
@@ -2217,61 +2217,50 @@ get_machine_name (unsigned e_machine)
     case EM_860:               return "Intel 80860";
     case EM_MIPS:              return "MIPS R3000";
     case EM_S370:              return "IBM System/370";
+      /* 10 */
     case EM_MIPS_RS3_LE:       return "MIPS R4000 big-endian";
     case EM_OLD_SPARCV9:       return "Sparc v9 (old)";
     case EM_PARISC:            return "HPPA";
-    case EM_PPC_OLD:           return "Power PC (old)";
+    case EM_VPP550:            return "Fujitsu VPP500";
     case EM_SPARC32PLUS:       return "Sparc v8+" ;
     case EM_960:               return "Intel 90860";
     case EM_PPC:               return "PowerPC";
+      /* 20 */
     case EM_PPC64:             return "PowerPC64";
+    case EM_S390_OLD:
+    case EM_S390:              return "IBM S/390";
+    case EM_SPU:               return "SPU";
+      /* 30 */
+    case EM_V800:              return "Renesas V850 (using RH850 ABI)";
     case EM_FR20:              return "Fujitsu FR20";
-    case EM_FT32:              return "FTDI FT32";
     case EM_RH32:              return "TRW RH32";
     case EM_MCORE:             return "MCORE";
+      /* 40 */
     case EM_ARM:               return "ARM";
     case EM_OLD_ALPHA:         return "Digital Alpha (old)";
     case EM_SH:                        return "Renesas / SuperH SH";
     case EM_SPARCV9:           return "Sparc v9";
     case EM_TRICORE:           return "Siemens Tricore";
     case EM_ARC:               return "ARC";
-    case EM_ARC_COMPACT:       return "ARCompact";
-    case EM_ARC_COMPACT2:      return "ARCv2";
     case EM_H8_300:            return "Renesas H8/300";
     case EM_H8_300H:           return "Renesas H8/300H";
     case EM_H8S:               return "Renesas H8S";
     case EM_H8_500:            return "Renesas H8/500";
+      /* 50 */
     case EM_IA_64:             return "Intel IA-64";
     case EM_MIPS_X:            return "Stanford MIPS-X";
     case EM_COLDFIRE:          return "Motorola Coldfire";
-    case EM_ALPHA:             return "Alpha";
-    case EM_CYGNUS_D10V:
-    case EM_D10V:              return "d10v";
-    case EM_CYGNUS_D30V:
-    case EM_D30V:              return "d30v";
-    case EM_CYGNUS_M32R:
-    case EM_M32R:              return "Renesas M32R (formerly Mitsubishi M32r)";
-    case EM_CYGNUS_V850:
-    case EM_V800:              return "Renesas V850 (using RH850 ABI)";
-    case EM_V850:              return "Renesas V850";
-    case EM_CYGNUS_MN10300:
-    case EM_MN10300:           return "mn10300";
-    case EM_CYGNUS_MN10200:
-    case EM_MN10200:           return "mn10200";
-    case EM_MOXIE:             return "Moxie";
-    case EM_CYGNUS_FR30:
-    case EM_FR30:              return "Fujitsu FR30";
-    case EM_CYGNUS_FRV:                return "Fujitsu FR-V";
-    case EM_PJ_OLD:
-    case EM_PJ:                        return "picoJava";
+    case EM_68HC12:            return "Motorola MC68HC12 Microcontroller";
     case EM_MMA:               return "Fujitsu Multimedia Accelerator";
     case EM_PCP:               return "Siemens PCP";
     case EM_NCPU:              return "Sony nCPU embedded RISC processor";
     case EM_NDR1:              return "Denso NDR1 microprocesspr";
     case EM_STARCORE:          return "Motorola Star*Core processor";
     case EM_ME16:              return "Toyota ME16 processor";
+      /* 60 */
     case EM_ST100:             return "STMicroelectronics ST100 processor";
     case EM_TINYJ:             return "Advanced Logic Corp. TinyJ embedded processor";
+    case EM_X86_64:            return "Advanced Micro Devices X86-64";
     case EM_PDSP:              return "Sony DSP processor";
     case EM_PDP10:             return "Digital Equipment Corp. PDP-10";
     case EM_PDP11:             return "Digital Equipment Corp. PDP-11";
@@ -2279,75 +2268,89 @@ get_machine_name (unsigned e_machine)
     case EM_ST9PLUS:           return "STMicroelectronics ST9+ 8/16 bit microcontroller";
     case EM_ST7:               return "STMicroelectronics ST7 8-bit microcontroller";
     case EM_68HC16:            return "Motorola MC68HC16 Microcontroller";
-    case EM_68HC12:            return "Motorola MC68HC12 Microcontroller";
+      /* 70 */
     case EM_68HC11:            return "Motorola MC68HC11 Microcontroller";
     case EM_68HC08:            return "Motorola MC68HC08 Microcontroller";
     case EM_68HC05:            return "Motorola MC68HC05 Microcontroller";
     case EM_SVX:               return "Silicon Graphics SVx";
     case EM_ST19:              return "STMicroelectronics ST19 8-bit microcontroller";
     case EM_VAX:               return "Digital VAX";
-    case EM_VISIUM:            return "CDS VISIUMcore processor";
-    case EM_AVR_OLD:
-    case EM_AVR:               return "Atmel AVR 8-bit microcontroller";
     case EM_CRIS:              return "Axis Communications 32-bit embedded processor";
     case EM_JAVELIN:           return "Infineon Technologies 32-bit embedded cpu";
     case EM_FIREPATH:          return "Element 14 64-bit DSP processor";
     case EM_ZSP:               return "LSI Logic's 16-bit DSP processor";
+      /* 80 */
     case EM_MMIX:              return "Donald Knuth's educational 64-bit processor";
     case EM_HUANY:             return "Harvard Universitys's machine-independent object format";
     case EM_PRISM:             return "Vitesse Prism";
-    case EM_X86_64:            return "Advanced Micro Devices X86-64";
-    case EM_L1OM:              return "Intel L1OM";
-    case EM_K1OM:              return "Intel K1OM";
-    case EM_S390_OLD:
-    case EM_S390:              return "IBM S/390";
-    case EM_SCORE:             return "SUNPLUS S+Core";
-    case EM_XSTORMY16:         return "Sanyo XStormy16 CPU core";
+    case EM_AVR_OLD:
+    case EM_AVR:               return "Atmel AVR 8-bit microcontroller";
+    case EM_CYGNUS_FR30:
+    case EM_FR30:              return "Fujitsu FR30";
+    case EM_CYGNUS_D10V:
+    case EM_D10V:              return "d10v";
+    case EM_CYGNUS_D30V:
+    case EM_D30V:              return "d30v";
+    case EM_CYGNUS_V850:
+    case EM_V850:              return "Renesas V850";
+    case EM_CYGNUS_M32R:
+    case EM_M32R:              return "Renesas M32R (formerly Mitsubishi M32r)";
+    case EM_CYGNUS_MN10300:
+    case EM_MN10300:           return "mn10300";
+      /* 90 */
+    case EM_CYGNUS_MN10200:
+    case EM_MN10200:           return "mn10200";
+    case EM_PJ:                        return "picoJava";
     case EM_OR1K:              return "OpenRISC 1000";
-    case EM_CRX:               return "National Semiconductor CRX microprocessor";
-    case EM_ADAPTEVA_EPIPHANY: return "Adapteva EPIPHANY";
-    case EM_DLX:               return "OpenDLX";
-    case EM_IP2K_OLD:
-    case EM_IP2K:              return "Ubicom IP2xxx 8-bit microcontrollers";
-    case EM_IQ2000:            return "Vitesse IQ2000";
+    case EM_ARC_COMPACT:       return "ARCompact";
     case EM_XTENSA_OLD:
     case EM_XTENSA:            return "Tensilica Xtensa Processor";
     case EM_VIDEOCORE:         return "Alphamosaic VideoCore processor";
     case EM_TMM_GPP:           return "Thompson Multimedia General Purpose Processor";
     case EM_NS32K:             return "National Semiconductor 32000 series";
     case EM_TPC:               return "Tenor Network TPC processor";
-    case EM_ST200:             return "STMicroelectronics ST200 microcontroller";
+    case EM_SNP1K:             return "Trebia SNP 1000 processor";
+      /* 100 */
+    case EM_ST200:             return "STMicroelectronics ST200 microcontroller";  
+    case EM_IP2K_OLD:
+    case EM_IP2K:              return "Ubicom IP2xxx 8-bit microcontrollers";
     case EM_MAX:               return "MAX Processor";
     case EM_CR:                        return "National Semiconductor CompactRISC";
     case EM_F2MC16:            return "Fujitsu F2MC16";
     case EM_MSP430:            return "Texas Instruments msp430 microcontroller";
-    case EM_LATTICEMICO32:     return "Lattice Mico32";
-    case EM_M32C_OLD:
-    case EM_M32C:              return "Renesas M32c";
-    case EM_MT:                 return "Morpho Techologies MT processor";
     case EM_BLACKFIN:          return "Analog Devices Blackfin";
     case EM_SE_C33:            return "S1C33 Family of Seiko Epson processors";
     case EM_SEP:               return "Sharp embedded microprocessor";
     case EM_ARCA:              return "Arca RISC microprocessor";
+      /* 110 */
     case EM_UNICORE:           return "Unicore";
     case EM_EXCESS:            return "eXcess 16/32/64-bit configurable embedded CPU";
     case EM_DXP:               return "Icera Semiconductor Inc. Deep Execution Processor";
-    case EM_NIOS32:            return "Altera Nios";
     case EM_ALTERA_NIOS2:      return "Altera Nios II";
+    case EM_CRX:               return "National Semiconductor CRX microprocessor";
+    case EM_XGATE:             return "Motorola XGATE embedded processor";
     case EM_C166:
     case EM_XC16X:             return "Infineon Technologies xc16x";
     case EM_M16C:              return "Renesas M16C series microprocessors";
     case EM_DSPIC30F:          return "Microchip Technology dsPIC30F Digital Signal Controller";
     case EM_CE:                        return "Freescale Communication Engine RISC core";
+      /* 120 */
+    case EM_M32C:              return "Renesas M32c";
+      /* 130 */
     case EM_TSK3000:           return "Altium TSK3000 core";
     case EM_RS08:              return "Freescale RS08 embedded processor";
     case EM_ECOG2:             return "Cyan Technology eCOG2 microprocessor";
+    case EM_SCORE:             return "SUNPLUS S+Core";
     case EM_DSP24:             return "New Japan Radio (NJR) 24-bit DSP Processor";
     case EM_VIDEOCORE3:                return "Broadcom VideoCore III processor";
+    case EM_LATTICEMICO32:     return "Lattice Mico32";
     case EM_SE_C17:            return "Seiko Epson C17 family";
+      /* 140 */
     case EM_TI_C6000:          return "Texas Instruments TMS320C6000 DSP family";
     case EM_TI_C2000:          return "Texas Instruments TMS320C2000 DSP family";
     case EM_TI_C5500:          return "Texas Instruments TMS320C55x DSP family";
+    case EM_TI_PRU:            return "TI PRU I/O processor";
+      /* 160 */
     case EM_MMDSP_PLUS:                return "STMicroelectronics 64bit VLIW Data Signal Processor";
     case EM_CYPRESS_M8C:       return "Cypress M8C microprocessor";
     case EM_R32C:              return "Renesas R32C series microprocessors";
@@ -2358,38 +2361,47 @@ get_machine_name (unsigned e_machine)
     case EM_NDS32:             return "Andes Technology compact code size embedded RISC processor family";
     case EM_ECOG1X:            return "Cyan Technology eCOG1X family";
     case EM_MAXQ30:            return "Dallas Semiconductor MAXQ30 Core microcontrollers";
+      /* 170 */
     case EM_XIMO16:            return "New Japan Radio (NJR) 16-bit DSP Processor";
     case EM_MANIK:             return "M2000 Reconfigurable RISC Microprocessor";
     case EM_CRAYNV2:           return "Cray Inc. NV2 vector architecture";
-    case EM_CYGNUS_MEP:         return "Toshiba MeP Media Engine";
-    case EM_CR16:
-    case EM_MICROBLAZE:
-    case EM_MICROBLAZE_OLD:    return "Xilinx MicroBlaze";
-    case EM_RISCV:             return "RISC-V";
-    case EM_RL78:              return "Renesas RL78";
     case EM_RX:                        return "Renesas RX";
     case EM_METAG:             return "Imagination Technologies Meta processor architecture";
     case EM_MCST_ELBRUS:       return "MCST Elbrus general purpose hardware architecture";
     case EM_ECOG16:            return "Cyan Technology eCOG16 family";
+    case EM_CR16:
+    case EM_MICROBLAZE:
+    case EM_MICROBLAZE_OLD:    return "Xilinx MicroBlaze";
     case EM_ETPU:              return "Freescale Extended Time Processing Unit";
     case EM_SLE9X:             return "Infineon Technologies SLE9X core";
-    case EM_AVR32:             return "Atmel Corporation 32-bit microprocessor family";
+      /* 180 */
+    case EM_L1OM:              return "Intel L1OM";
+    case EM_K1OM:              return "Intel K1OM";
+    case EM_INTEL182:          return "Intel (reserved)";
+    case EM_AARCH64:           return "AArch64";
+    case EM_ARM184:            return "ARM (reserved)";
+    case EM_AVR32:             return "Atmel Corporation 32-bit microprocessor";
     case EM_STM8:              return "STMicroeletronics STM8 8-bit microcontroller";
     case EM_TILE64:            return "Tilera TILE64 multicore architecture family";
     case EM_TILEPRO:           return "Tilera TILEPro multicore architecture family";
-    case EM_TILEGX:            return "Tilera TILE-Gx multicore architecture family";
+      /* 190 */
     case EM_CUDA:              return "NVIDIA CUDA architecture";
-    case EM_XGATE:             return "Motorola XGATE embedded processor";
+    case EM_TILEGX:            return "Tilera TILE-Gx multicore architecture family";
     case EM_CLOUDSHIELD:       return "CloudShield architecture family";
     case EM_COREA_1ST:         return "KIPO-KAIST Core-A 1st generation processor family";
     case EM_COREA_2ND:         return "KIPO-KAIST Core-A 2nd generation processor family";
+    case EM_ARC_COMPACT2:      return "ARCv2";
     case EM_OPEN8:             return "Open8 8-bit RISC soft processor core";
+    case EM_RL78:              return "Renesas RL78";
     case EM_VIDEOCORE5:                return "Broadcom VideoCore V processor";
+    case EM_78K0R:             return "Renesas 78K0R";
+      /* 200 */
     case EM_56800EX:           return "Freescale 56800EX Digital Signal Controller (DSC)";
     case EM_BA1:               return "Beyond BA1 CPU architecture";
     case EM_BA2:               return "Beyond BA2 CPU architecture";
     case EM_XCORE:             return "XMOS xCORE processor family";
     case EM_MCHP_PIC:          return "Microchip 8-bit PIC(r) family";
+      /* 210 */
     case EM_KM32:              return "KM211 KM32 32-bit processor";
     case EM_KMX32:             return "KM211 KMX32 32-bit processor";
     case EM_KMX16:             return "KM211 KMX16 16-bit processor";
@@ -2400,9 +2412,29 @@ get_machine_name (unsigned e_machine)
     case EM_COOL:              return "Bluechip Systems CoolEngine";
     case EM_NORC:              return "Nanoradio Optimized RISC";
     case EM_CSR_KALIMBA:       return "CSR Kalimba architecture family";
+      /* 220 */
     case EM_Z80:               return "Zilog Z80";
-    case EM_AMDGPU:            return "AMD GPU architecture";
-    case EM_TI_PRU:            return "TI PRU I/O processor";
+    case EM_VISIUM:            return "CDS VISIUMcore processor";
+    case EM_FT32:               return "FTDI Chip FT32";
+    case EM_MOXIE:              return "Moxie";
+    case EM_AMDGPU:            return "AMD GPU";
+    case EM_RISCV:             return "RISC-V";
+    case EM_LANAI:             return "Lanai 32-bit processor";
+    case EM_BPF:               return "Linux BPF";
+
+      /* Large numbers...  */
+    case EM_MT:                 return "Morpho Techologies MT processor";
+    case EM_ALPHA:             return "Alpha";
+    case EM_WEBASSEMBLY:       return "Web Assembly";
+    case EM_DLX:               return "OpenDLX";  
+    case EM_XSTORMY16:         return "Sanyo XStormy16 CPU core";
+    case EM_IQ2000:            return "Vitesse IQ2000";
+    case EM_M32C_OLD:
+    case EM_NIOS32:            return "Altera Nios";
+    case EM_CYGNUS_MEP:         return "Toshiba MeP Media Engine";
+    case EM_ADAPTEVA_EPIPHANY: return "Adapteva EPIPHANY";
+    case EM_CYGNUS_FRV:                return "Fujitsu FR-V";
+
     default:
       snprintf (buff, sizeof (buff), _("<unknown>: 0x%x"), e_machine);
       return buff;
@@ -15926,11 +15958,11 @@ get_gnu_elf_note_type (unsigned e_type)
 }
 
 static void
-decode_x86_isa (unsigned long bitmask)
+decode_x86_isa (unsigned int bitmask)
 {
   while (bitmask)
     {
-      unsigned long bit = bitmask & (- bitmask);
+      unsigned int bit = bitmask & (- bitmask);
 
       bitmask &= ~ bit;
       switch (bit)
@@ -15953,7 +15985,7 @@ decode_x86_isa (unsigned long bitmask)
        case GNU_PROPERTY_X86_ISA_1_AVX512VL: printf ("AVX512VL"); break;
        case GNU_PROPERTY_X86_ISA_1_AVX512DQ: printf ("AVX512DQ"); break;
        case GNU_PROPERTY_X86_ISA_1_AVX512BW: printf ("AVX512BW"); break;
-       default: printf (_("<unknown: %lx>"), bit); break;
+       default: printf (_("<unknown: %x>"), bit); break;
        }
       if (bitmask)
        printf (", ");
@@ -15969,73 +16001,106 @@ print_gnu_property_note (Elf_Internal_Note * pnote)
 
   printf (_("      Properties: "));
 
-  if (pnote->descsz % size)
+  if (pnote->descsz < 8 || (pnote->descsz % size) != 0)
     {
       printf (_("<corrupt GNU_PROPERTY_TYPE, size = %#lx>\n"), pnote->descsz);
       return;
     }
 
-  while (ptr < (ptr_end - (size * 2)))
+  while (1)
     {
-      unsigned long j;
-      unsigned long type = byte_get (ptr, size);
-      unsigned long datasz = byte_get (ptr + size, size);
+      unsigned int j;
+      unsigned int type = byte_get (ptr, 4);
+      unsigned int datasz = byte_get (ptr + 4, 4);
 
-      ptr += 2 * size;
+      ptr += 8;
 
-      switch (type)
+      if ((ptr + datasz) > ptr_end)
        {
-       case GNU_PROPERTY_STACK_SIZE:
-         printf (_("stack size: "));
-         if (datasz != size || (ptr + size  > ptr_end))
-           printf (_("<corrupt length: %#lx> "), datasz);
-         else
-           printf ("%#lx", (unsigned long) byte_get (ptr, size));
-         break;
-
-       case GNU_PROPERTY_NO_COPY_ON_PROTECTED:
-         printf ("no copy on protected ");
-         if (datasz)
-           printf (_("<corrupt length: %#lx> "), datasz);
+         printf (_("<corrupt type (%#x) datasz: %#x>\n"),
+                 type, datasz);
          break;
+       }
 
-       case GNU_PROPERTY_X86_ISA_1_USED:
-         printf ("x86 ISA used: ");
-         if (datasz != size  || (ptr + size > ptr_end))
-           printf (_("<corrupt length: %#lx> "), datasz);
-         else
-           decode_x86_isa (byte_get (ptr, size));
-         break;
+      if (type >= GNU_PROPERTY_LOPROC && type <= GNU_PROPERTY_HIPROC)
+       {
+         if (elf_header.e_machine == EM_X86_64
+             || elf_header.e_machine == EM_IAMCU
+             || elf_header.e_machine == EM_386)
+           {
+             switch (type)
+               {
+               case GNU_PROPERTY_X86_ISA_1_USED:
+                 printf ("x86 ISA used: ");
+                 if (datasz != 4)
+                   printf (_("<corrupt length: %#x> "), datasz);
+                 else
+                   decode_x86_isa (byte_get (ptr, 4));
+                 goto next;
 
-       case GNU_PROPERTY_X86_ISA_1_NEEDED:
-         printf ("x86 ISA needed: ");
-         if (datasz != size  || (ptr + size > ptr_end))
-           printf (_("<corrupt length: %#lx> "), datasz);
-         else
-           decode_x86_isa (byte_get (ptr, size));
-         break;
+               case GNU_PROPERTY_X86_ISA_1_NEEDED:
+                 printf ("x86 ISA needed: ");
+                 if (datasz != 4)
+                   printf (_("<corrupt length: %#x> "), datasz);
+                 else
+                   decode_x86_isa (byte_get (ptr, 4));
+                 goto next;
 
-       default:
-         printf (_("<unknown type %#lx data: "), type);
-         if (ptr + datasz > ptr_end)
+               default:
+                 break;
+               }
+           }
+       }
+      else
+       {
+         switch (type)
            {
-             printf (_("corrupt datasz: %#lx>\n"), datasz);
+           case GNU_PROPERTY_STACK_SIZE:
+             printf (_("stack size: "));
+             if (datasz != size)
+               printf (_("<corrupt length: %#x> "), datasz);
+             else
+               printf ("%#lx", (unsigned long) byte_get (ptr, size));
+             goto next;
+
+           case GNU_PROPERTY_NO_COPY_ON_PROTECTED:
+             printf ("no copy on protected ");
+             if (datasz)
+               printf (_("<corrupt length: %#x> "), datasz);
+             goto next;
+
+           default:
              break;
            }
-         for (j = 0; j < datasz; ++j)
-           printf ("%02x ", ptr[j] & 0xff);
-         printf (">");
-         break;
        }
 
+      if (type < GNU_PROPERTY_LOPROC)
+       printf (_("<unknown type %#x data: "), type);
+      else if (type < GNU_PROPERTY_LOUSER)
+       printf (_("<procesor-specific type %#x data: "), type);
+      else
+       printf (_("<application-specific type %#x data: "), type);
+      for (j = 0; j < datasz; ++j)
+       printf ("%02x ", ptr[j] & 0xff);
+      printf (">");
+
+next:
       ptr += ((datasz + (size - 1)) & ~ (size - 1));
-      if (ptr < (ptr_end - (size * 2)))
+      if (ptr == ptr_end)
+       break;
+      else
        {
          if (do_wide)
            printf (", ");
          else
            printf ("\n\t");
        }
+
+      if (ptr > (ptr_end - 8))
+       {
+         printf (_("<corrupt descsz: %#lx>\n"), pnote->descsz);
+         break;
+       }
     }
 
   printf ("\n");
@@ -16533,99 +16598,169 @@ print_ia64_vms_note (Elf_Internal_Note * pnote)
   return TRUE;
 }
 
+/* Print the name of the symbol associated with a build attribute
+   that is attached to address OFFSET.  */
+
 static bfd_boolean
-print_gnu_build_attribute_description (Elf_Internal_Note * pnote,
-                                      FILE *              file,
-                                      Elf_Internal_Shdr * section ATTRIBUTE_UNUSED)
+print_symbol_for_build_attribute (FILE *         file,
+                                 unsigned long  offset,
+                                 bfd_boolean    is_open_attr)
 {
-  static unsigned long global_offset = 0;
-  unsigned long       i;
-  unsigned long       strtab_size = 0;
-  char *              strtab = NULL;
-  Elf_Internal_Sym *  symtab = NULL;
-  unsigned long       nsyms = 0;
-  Elf_Internal_Shdr * symsec = NULL;
-  unsigned int        desc_size = is_32bit_elf ? 4 : 8;
+  static FILE *             saved_file = NULL;
+  static char *             strtab;
+  static unsigned long      strtablen;
+  static Elf_Internal_Sym * symtab;
+  static unsigned long      nsyms;
+  Elf_Internal_Sym *  saved_sym = NULL;
+  Elf_Internal_Sym *  sym;
 
-  if (pnote->descsz  == 0)
+  if (saved_file == NULL || file != saved_file)
     {
-      printf (_("    Applies from offset %#lx\n"), global_offset);
-      return TRUE;
-    }
-
-  if (pnote->descsz != desc_size)
-    {
-      error (_("    <invalid description size: %lx>\n"), pnote->descsz);
-      printf (_("    <invalid descsz>"));
-      return FALSE;
-    }
+      Elf_Internal_Shdr * symsec;
 
-  /* Load the symbols.  */
-  for (symsec = section_headers;
-       symsec < section_headers + elf_header.e_shnum;
-       symsec ++)
-    {
-      if (symsec->sh_type == SHT_SYMTAB)
+      /* Load the symbol and string sections.  */
+      for (symsec = section_headers;
+          symsec < section_headers + elf_header.e_shnum;
+          symsec ++)
        {
-         symtab = GET_ELF_SYMBOLS (file, symsec, & nsyms);
-
-         if (symsec->sh_link < elf_header.e_shnum)
+         if (symsec->sh_type == SHT_SYMTAB)
            {
-             Elf_Internal_Shdr * strtab_sec = section_headers + symsec->sh_link;
+             symtab = GET_ELF_SYMBOLS (file, symsec, & nsyms);
 
-             strtab = (char *) get_data (NULL, file, strtab_sec->sh_offset,
-                                         1, strtab_sec->sh_size,
-                                         _("string table"));
-             strtab_size = strtab != NULL ? strtab_sec->sh_size : 0;
+             if (symsec->sh_link < elf_header.e_shnum)
+               {
+                 Elf_Internal_Shdr * strtab_sec = section_headers + symsec->sh_link;
+
+                 strtab = (char *) get_data (NULL, file, strtab_sec->sh_offset,
+                                             1, strtab_sec->sh_size,
+                                             _("string table"));
+                 strtablen = strtab != NULL ? strtab_sec->sh_size : 0;
+               }
            }
        }
+      saved_file = file;
     }
 
-  printf (_("    Applies from offset"));
-
-  for (i = 0; i < pnote->descsz; i += desc_size)
+  if (symtab == NULL || strtab == NULL)
     {
-      Elf_Internal_Sym * sym;
-      unsigned long offset;
+      printf ("\n");
+      return FALSE;
+    }
 
-      offset = byte_get ((unsigned char *) pnote->descdata + i, desc_size);
+  /* Find a symbol whose value matches offset.  */
+  for (sym = symtab; sym < symtab + nsyms; sym ++)
+    if (sym->st_value == offset)
+      {
+       if (sym->st_name >= strtablen)
+         /* Huh ?  This should not happen.  */
+         continue;
 
-      if (i + desc_size == pnote->descsz)
-       printf (_(" %#lx"), offset);
-      else
-       printf (_(" %#lx, "), offset);
+       if (strtab[sym->st_name] == 0)
+         continue;
+
+       if (is_open_attr)
+         {
+           /* For OPEN attributes we prefer GLOBAL over LOCAL symbols
+              and FILE or OBJECT symbols over NOTYPE symbols.  We skip
+              FUNC symbols entirely.  */
+           switch (ELF_ST_TYPE (sym->st_info))
+             {
+             case STT_FILE:
+               saved_sym = sym;
+               /* We can stop searching now.  */
+               sym = symtab + nsyms;
+               continue;
 
-      if (pnote->type == NT_GNU_BUILD_ATTRIBUTE_OPEN)
-       global_offset = offset;
+             case STT_OBJECT:
+               saved_sym = sym;
+               continue;
 
-      if (symtab == NULL || strtab == NULL)
-       continue;
+             case STT_FUNC:
+               /* Ignore function symbols.  */
+               continue;
 
-      /* Find a symbol whose value matches offset.  */
-      for (sym = symtab; sym < symtab + nsyms; sym ++)
-       if (sym->st_value == offset)
-         {
-           if (sym->st_name < strtab_size)
+             default:
+               break;
+             }
+
+           switch (ELF_ST_BIND (sym->st_info))
              {
-               if (strtab[sym->st_name] == 0)
-                 continue;
-               if (pnote->type == NT_GNU_BUILD_ATTRIBUTE_OPEN)
-                 printf (_(" (file: %s)"), strtab + sym->st_name);
-               else if (ELF_ST_TYPE (sym->st_info) != STT_FUNC)
-                 continue;
-               else
-                 printf (_(" (function: %s)"), strtab + sym->st_name);
+             case STB_GLOBAL:
+               if (saved_sym == NULL
+                   || ELF_ST_TYPE (saved_sym->st_info) != STT_OBJECT)
+                 saved_sym = sym;
+               break;
+
+             case STB_LOCAL:
+               if (saved_sym == NULL)
+                 saved_sym = sym;
+               break;
+
+             default:
                break;
              }
          }
-      if (sym == symtab + nsyms)
-       printf (_(" (<symbol name unknown>)"));
-    }
+       else
+         {
+           if (ELF_ST_TYPE (sym->st_info) != STT_FUNC)
+             continue;
 
-  printf ("\n");
+           saved_sym = sym;
+           break;
+         }
+      }
+
+  printf (" (%s: %s)\n",
+         is_open_attr ? _("file") : _("func"),
+         saved_sym ? strtab + saved_sym->st_name : _("<no symbol found>)"));
   return TRUE;
 }
 
+static bfd_boolean
+print_gnu_build_attribute_description (Elf_Internal_Note * pnote,
+                                      FILE *              file)
+{
+  static unsigned long global_offset = 0;
+  unsigned long        offset;
+  unsigned int         desc_size = is_32bit_elf ? 4 : 8;
+  bfd_boolean          is_open_attr = pnote->type == NT_GNU_BUILD_ATTRIBUTE_OPEN;
+
+  if (pnote->descsz == 0)
+    {
+      if (is_open_attr)
+       {
+         printf (_("    Applies from offset %#lx\n"), global_offset);
+         return TRUE;
+       }
+      else
+       {
+         printf (_("    Applies to func at %#lx"), global_offset);
+         return print_symbol_for_build_attribute (file, global_offset, is_open_attr);
+       }
+    }
+
+  if (pnote->descsz != desc_size)
+    {
+      error (_("    <invalid description size: %lx>\n"), pnote->descsz);
+      printf (_("    <invalid descsz>"));
+      return FALSE;
+    }
+
+  offset = byte_get ((unsigned char *) pnote->descdata, desc_size);
+
+  if (is_open_attr)
+    {
+      printf (_("    Applies from offset %#lx"), offset);
+      global_offset = offset;
+    }
+  else
+    {
+      printf (_("    Applies to func at %#lx"), offset);
+    }
+
+  return print_symbol_for_build_attribute (file, offset, is_open_attr);
+}
+
 static bfd_boolean
 print_gnu_build_attribute_name (Elf_Internal_Note * pnote)
 {
@@ -16750,7 +16885,9 @@ print_gnu_build_attribute_name (Elf_Internal_Note * pnote)
 
        while (bytes --)
          {
-           val |= ((* name ++) << shift);
+           unsigned long byte = (* name ++) & 0xff;
+
+           val |= byte << shift;
            shift += 8;
          }
 
@@ -16808,8 +16945,7 @@ print_gnu_build_attribute_name (Elf_Internal_Note * pnote)
 
 static bfd_boolean
 process_note (Elf_Internal_Note *  pnote,
-             FILE *               file,
-             Elf_Internal_Shdr *  section)
+             FILE *               file)
 {
   const char * name = pnote->namesz ? pnote->namedata : "(NONE)";
   const char * nt;
@@ -16877,7 +17013,7 @@ process_note (Elf_Internal_Note *  pnote,
     return print_core_note (pnote);
   else if (pnote->type == NT_GNU_BUILD_ATTRIBUTE_OPEN
           || pnote->type == NT_GNU_BUILD_ATTRIBUTE_FUNC)
-    return print_gnu_build_attribute_description (pnote, file, section);
+    return print_gnu_build_attribute_description (pnote, file);
 
   if (pnote->descsz)
     {
@@ -17024,14 +17160,14 @@ process_notes_at (FILE *              file,
              break;
            }
 
-         strncpy (temp, inote.namedata, inote.namesz);
+         memcpy (temp, inote.namedata, inote.namesz);
          temp[inote.namesz] = 0;
 
          /* warn (_("'%s' NOTE name not properly null terminated\n"), temp);  */
          inote.namedata = temp;
        }
 
-      if (! process_note (& inote, file, section))
+      if (! process_note (& inote, file))
        res = FALSE;
 
       if (temp != NULL)