source: patches/binutils-2.19-branch_update-1.patch@ a210462

bfd/ChangeLog

@@ +1 @@
+2008-11-20  Alan Modra  <amodra@bigpond.net.au>
+
+        * elf32-ppc.c (allocate_dynrelocs): Always use tlsld_got for
+        TLS_LD even when symbol is used with other TLS reloc types.
+        (ppc_elf_relocate_section): Bypass symbol checks when using tlsld_got.
+        Leave addend zero on LD DTPMOD dynamic reloc.
+
+2008-11-17  Eric B. Weddington  <eric.weddington@atmel.com>
+
+        PR 7022
+        * elf32-avr.c (bfd_elf_avr_final_write_processing):
+        Add missing break statements.
+
+2008-10-28  Tristan Gingold  <gingold@adacore.com>
+
+        * configure.in: Bump version to 2.19.0
+        * Makefile.am (RELEASE): Unset.
+        * configure, Makefile.in: Regenerated.
+
 2008-10-16  Tristan Gingold  <gingold@adacore.com>
 
         * configure.in: Bump version to 2.19

bfd/Makefile.in

@@ -271 +271 @@
 ACLOCAL_AMFLAGS = -I . -I .. -I ../config
 
 # Uncomment the following line when doing a release.
-RELEASE = y
+RELEASE=y
 INCDIR = $(srcdir)/../include
 CSEARCH = -I. -I$(srcdir) -I$(INCDIR)
 MKDEP = gcc -MM

bfd/configure

@@ -3032 +3032 @@
 
 # Define the identity of the package.
  PACKAGE=bfd
- VERSION=2.19
+ VERSION=2.19.0
 
 
 cat >>confdefs.h <<_ACEOF

bfd/configure.in

@@ -8 +8 @@
 AC_CANONICAL_TARGET
 AC_ISC_POSIX
 
-AM_INIT_AUTOMAKE(bfd, 2.19)
+AM_INIT_AUTOMAKE(bfd, 2.19.0)
 
 dnl These must be called before LT_INIT, because it may want
 dnl to call AC_CHECK_PROG.

bfd/elf32-avr.c

@@ -1298 +1298 @@
 
     case bfd_mach_avr25:
       val = E_AVR_MACH_AVR25;
+      break;
 
     case bfd_mach_avr3:
       val = E_AVR_MACH_AVR3;
@@ -1305 +1306 @@
 
     case bfd_mach_avr31:
       val = E_AVR_MACH_AVR31;
+      break;
 
     case bfd_mach_avr35:
       val = E_AVR_MACH_AVR35;
+      break;
 
     case bfd_mach_avr4:
       val = E_AVR_MACH_AVR4;

bfd/elf32-ppc.c

@@ -4997 +4997 @@
   eh = (struct ppc_elf_link_hash_entry *) h;
   if (eh->elf.got.refcount > 0)
     {
+      bfd_boolean dyn;
+      unsigned int need;
+
       /* Make sure this symbol is output as a dynamic symbol.  */
       if (eh->elf.dynindx == -1
           && !eh->elf.forced_local
@@ -5006 +5009 @@
             return FALSE;
         }
 
-      if (eh->tls_mask == (TLS_TLS | TLS_LD)
-          && !eh->elf.def_dynamic)
-        {
-          /* If just an LD reloc, we'll just use htab->tlsld_got.offset.  */
-          htab->tlsld_got.refcount += 1;
-          eh->elf.got.offset = (bfd_vma) -1;
-        }
-      else
+      need = 0;
+      if ((eh->tls_mask & TLS_TLS) != 0)
         {
-          bfd_boolean dyn;
-          unsigned int need = 0;
-          if ((eh->tls_mask & TLS_TLS) != 0)
+          if ((eh->tls_mask & TLS_LD) != 0)
             {
-              if ((eh->tls_mask & TLS_LD) != 0)
-                need += 8;
-              if ((eh->tls_mask & TLS_GD) != 0)
+              if (!eh->elf.def_dynamic)
+                /* We'll just use htab->tlsld_got.offset.  This should
+                   always be the case.  It's a little odd if we have
+                   a local dynamic reloc against a non-local symbol.  */
+                htab->tlsld_got.refcount += 1;
+              else
                 need += 8;
-              if ((eh->tls_mask & (TLS_TPREL | TLS_TPRELGD)) != 0)
-                need += 4;
-              if ((eh->tls_mask & TLS_DTPREL) != 0)
-                need += 4;
             }
-          else
+          if ((eh->tls_mask & TLS_GD) != 0)
+            need += 8;
+          if ((eh->tls_mask & (TLS_TPREL | TLS_TPRELGD)) != 0)
+            need += 4;
+          if ((eh->tls_mask & TLS_DTPREL) != 0)
             need += 4;
+        }
+      else
+        need += 4;
+      if (need == 0)
+        eh->elf.got.offset = (bfd_vma) -1;
+      else
+        {
           eh->elf.got.offset = allocate_got (htab, need);
           dyn = htab->elf.dynamic_sections_created;
           if ((info->shared
@@ -5039 +5044 @@
             {
               /* All the entries we allocated need relocs.
                  Except LD only needs one.  */
-              if ((eh->tls_mask & TLS_LD) != 0)
+              if ((eh->tls_mask & TLS_LD) != 0
+                  && eh->elf.def_dynamic)
                 need -= 4;
               htab->relgot->size += need * (sizeof (Elf32_External_Rela) / 4);
             }
@@ -5275 +5281 @@
       for (; local_got < end_local_got; ++local_got, ++lgot_masks)
         if (*local_got > 0)
           {
-            if (*lgot_masks == (TLS_TLS | TLS_LD))
+            unsigned int need = 0;
+            if ((*lgot_masks & TLS_TLS) != 0)
               {
-                /* If just an LD reloc, we'll just use
-                   htab->tlsld_got.offset.  */
-                htab->tlsld_got.refcount += 1;
-                *local_got = (bfd_vma) -1;
+                if ((*lgot_masks & TLS_GD) != 0)
+                  need += 8;
+                if ((*lgot_masks & TLS_LD) != 0)
+                  htab->tlsld_got.refcount += 1;
+                if ((*lgot_masks & (TLS_TPREL | TLS_TPRELGD)) != 0)
+                  need += 4;
+                if ((*lgot_masks & TLS_DTPREL) != 0)
+                  need += 4;
               }
             else
+              need += 4;
+            if (need == 0)
+              *local_got = (bfd_vma) -1;
+            else
               {
-                unsigned int need = 0;
-                if ((*lgot_masks & TLS_TLS) != 0)
-                  {
-                    if ((*lgot_masks & TLS_GD) != 0)
-                      need += 8;
-                    if ((*lgot_masks & (TLS_TPREL | TLS_TPRELGD)) != 0)
-                      need += 4;
-                    if ((*lgot_masks & TLS_DTPREL) != 0)
-                      need += 4;
-                  }
-                else
-                  need += 4;
                 *local_got = allocate_got (htab, need);
                 if (info->shared)
                   htab->relgot->size += (need
@@ -6560 +6563 @@
 
                     /* Generate relocs for the dynamic linker.  */
                     if ((info->shared || indx != 0)
-                        && (h == NULL
+                        && (offp == &htab->tlsld_got.offset
+                            || h == NULL
                             || ELF_ST_VISIBILITY (h->other) == STV_DEFAULT
                             || h->root.type != bfd_link_hash_undefweak))
                       {
@@ -6591 +6595 @@
                           outrel.r_info = ELF32_R_INFO (indx, R_PPC_RELATIVE);
                         else
                           outrel.r_info = ELF32_R_INFO (indx, R_PPC_GLOB_DAT);
-                        if (indx == 0)
+                        if (indx == 0 && tls_ty != (TLS_TLS | TLS_LD))
                           {
                             outrel.r_addend += relocation;
                             if (tls_ty & (TLS_GD | TLS_DTPREL | TLS_TPREL))

bfd/version.h

@@ -1 @@
-#define BFD_VERSION_DATE 20081016
+#define BFD_VERSION_DATE 20081220
 #define BFD_VERSION @bfd_version@
 #define BFD_VERSION_STRING  @bfd_version_package@ @bfd_version_string@
 #define REPORT_BUGS_TO @report_bugs_to@

binutils/ChangeLog

@@ +1 @@
+2008-12-01  Nick Clifton  <nickc@redhat.com>
+
+        PR 7044
+        * dlltool.c (run): Use formatting string to avoid compile time
+        warning.
+        (gen_exp_file): Check return value from fread.
+        * windmc.c (main): Check return value from fread.
+
 2008-09-25  Richard Henderson  <rth@redhat.com>
 
         * dwarf.c (size_of_encoded_value, get_encoded_value): Move up.

binutils/dlltool.c

@@ -1206 +1206 @@
 
   if (pid == -1)
     {
-      inform (strerror (errno));
+      inform ("%s", strerror (errno));
 
       fatal (errmsg_fmt, errmsg_arg);
     }
@@ -1992 +1992 @@
       numbytes = ftell (base_file);
       fseek (base_file, 0, SEEK_SET);
       copy = xmalloc (numbytes);
-      fread (copy, 1, numbytes, base_file);
+      if (fread (copy, 1, numbytes, base_file) < numbytes)
+        fatal (_("failed to read the number of entries from base file"));
       num_entries = numbytes / sizeof (long);
 
-
       fprintf (f, "\t.section\t.reloc\n");
       if (num_entries)
         {

binutils/windmc.c

@@ -1155 +1155 @@
     fseek (fp, 0, SEEK_SET);
     buff = malloc (flen + 3);
     memset (buff, 0, flen + 3);
-    fread (buff, 1, flen, fp);
+    if (fread (buff, 1, flen, fp) < flen)
+      fatal (_("unable to read contents of %s"), input_filename);
     fclose (fp);
     if (mcset_text_in_is_unicode != 1)
       {

configure.ac

@@ -166 +166 @@
 # binutils, gas and ld appear in that order because it makes sense to run
 # "make check" in that particular order.
 # If --enable-gold is used, "gold" will replace "ld".
-host_tools="byacc flex bison binutils gas ld fixincludes gcc sid sim gdb make patch prms send-pr gprof etc expect dejagnu ash bash bzip2 m4 autoconf automake libtool diff rcs fileutils shellutils time textutils wdiff find uudecode hello tar gzip indent recode release sed utils guile perl gawk findutils gettext zip fastjar gnattools"
+host_tools="texinfo byacc flex bison binutils gas ld fixincludes gcc sid sim gdb make patch prms send-pr gprof etc expect dejagnu ash bash bzip2 m4 autoconf automake libtool diff rcs fileutils shellutils time textutils wdiff find uudecode hello tar gzip indent recode release sed utils guile perl gawk findutils gettext zip fastjar gnattools"
 
 # libgcj represents the runtime libraries only used by gcj.
 libgcj="target-libffi \

ld/ChangeLog

@@ +1 @@
+2008-11-14  Alan Modra  <amodra@bigpond.net.au>
+
+        * Makefile.am (spu_ovl.o_c): Add missing line continuations.
+        * Makefile.in: Regenerate.
+
 2008-10-05  Alan Modra  <amodra@bigpond.net.au>
 
         PR 6943

ld/Makefile.am

@@ -758 +758 @@
 $(srcdir)/emultempl/spu_ovl.o_c: @MAINT@ $(srcdir)/emultempl/spu_ovl.S
         if ../gas/as-new --version \
                 | grep 'target.*spu' >/dev/null 2>/dev/null; then \
-          cpp -DOVLY_IRQ_SAVE $(srcdir)/emultempl/spu_ovl.S spu_ovl.s
+          cpp -DOVLY_IRQ_SAVE $(srcdir)/emultempl/spu_ovl.S spu_ovl.s; \
           ../gas/as-new -o spu_ovl.o spu_ovl.s; \
-          ../binutils/bin2c <spu_ovl.o >$@
+          ../binutils/bin2c <spu_ovl.o >$@; \
         fi
 eelf32_i860.c: $(srcdir)/emulparams/elf32_i860.sh \
   $(ELF_GEN_DEPS) $(srcdir)/scripttempl/elf.sc ${GEN_DEPENDS}

ld/Makefile.in

@@ -1588 +1588 @@
 $(srcdir)/emultempl/spu_ovl.o_c: @MAINT@ $(srcdir)/emultempl/spu_ovl.S
         if ../gas/as-new --version \
                 | grep 'target.*spu' >/dev/null 2>/dev/null; then \
-          cpp -DOVLY_IRQ_SAVE $(srcdir)/emultempl/spu_ovl.S spu_ovl.s
+          cpp -DOVLY_IRQ_SAVE $(srcdir)/emultempl/spu_ovl.S spu_ovl.s; \
           ../gas/as-new -o spu_ovl.o spu_ovl.s; \
-          ../binutils/bin2c <spu_ovl.o >$@
+          ../binutils/bin2c <spu_ovl.o >$@; \
         fi
 eelf32_i860.c: $(srcdir)/emulparams/elf32_i860.sh \
   $(ELF_GEN_DEPS) $(srcdir)/scripttempl/elf.sc ${GEN_DEPENDS}

opcodes/ChangeLog

@@ +1 @@
+2008-11-27  Alan Modra  <amodra@bigpond.net.au>
+
+        * ppc-opc.c (extract_sprg): Correct operand range check.
+
+2008-11-26  Andreas Schwab  <schwab@suse.de>
+
+        * m68k-dis.c (NEXTBYTE, NEXTWORD, NEXTLONG, NEXTULONG, NEXTSINGLE)
+        (NEXTDOUBLE, NEXTEXTEND, NEXTPACKED): Fix error handling.
+        (save_printer, save_print_address): Remove.
+        (fetch_data): Don't use them.
+        (match_insn_m68k): Always restore printing functions.
+        (print_insn_m68k): Don't save/restore printing functions.
+
+2008-11-25  Nick Clifton  <nickc@redhat.com>
+
+        * m68k-dis.c: Rewrite to remove use of setjmp/longjmp.
+
 2008-09-29  Nick Clifton  <nickc@redhat.com>
 
         * po/vi.po: Updated Vietnamese translation.

opcodes/m68k-dis.c

@@ -60 +60 @@
 #endif
 
 /* Get a 1 byte signed integer.  */
-#define NEXTBYTE(p)  (p += 2, FETCH_DATA (info, p), COERCE_SIGNED_CHAR(p[-1]))
+#define NEXTBYTE(p, val)                        \
+  do                                            \
+    {                                           \
+      p += 2;                                   \
+      if (!FETCH_DATA (info, p))                \
+        return -3;                              \
+      val = COERCE_SIGNED_CHAR (p[-1]);         \
+    }                                           \
+  while (0)
 
 /* Get a 2 byte signed integer.  */
 #define COERCE16(x) ((int) (((x) ^ 0x8000) - 0x8000))
-#define NEXTWORD(p)  \
-  (p += 2, FETCH_DATA (info, p), \
-   COERCE16 ((p[-2] << 8) + p[-1]))
+
+#define NEXTWORD(p, val, ret_val)               \
+  do                                            \
+    {                                           \
+      p += 2;                                   \
+      if (!FETCH_DATA (info, p))                \
+        return ret_val;                         \
+      val = COERCE16 ((p[-2] << 8) + p[-1]);    \
+    }                                           \
+  while (0)                                             
 
 /* Get a 4 byte signed integer.  */
 #define COERCE32(x) ((bfd_signed_vma) ((x) ^ 0x80000000) - 0x80000000)
-#define NEXTLONG(p)  \
-  (p += 4, FETCH_DATA (info, p), \
-   (COERCE32 ((((((p[-4] << 8) + p[-3]) << 8) + p[-2]) << 8) + p[-1])))
+
+#define NEXTLONG(p, val, ret_val)                                       \
+  do                                                                    \
+    {                                                                   \
+      p += 4;                                                           \
+      if (!FETCH_DATA (info, p))                                        \
+        return ret_val;                                                 \
+      val = COERCE32 ((((((p[-4] << 8) + p[-3]) << 8) + p[-2]) << 8) + p[-1]); \
+    }                                                                   \
+  while (0)
 
 /* Get a 4 byte unsigned integer.  */
-#define NEXTULONG(p)  \
-  (p += 4, FETCH_DATA (info, p), \
-   (unsigned int) ((((((p[-4] << 8) + p[-3]) << 8) + p[-2]) << 8) + p[-1]))
+#define NEXTULONG(p, val)                                               \
+  do                                                                    \
+    {                                                                   \
+      p += 4;                                                           \
+      if (!FETCH_DATA (info, p))                                        \
+        return -3;                                                      \
+      val = (unsigned int) ((((((p[-4] << 8) + p[-3]) << 8) + p[-2]) << 8) + p[-1]); \
+    }                                                                   \
+  while (0)
 
 /* Get a single precision float.  */
-#define NEXTSINGLE(val, p) \
-  (p += 4, FETCH_DATA (info, p), \
-   floatformat_to_double (&floatformat_ieee_single_big, (char *) p - 4, &val))
+#define NEXTSINGLE(val, p)                                      \
+  do                                                            \
+    {                                                           \
+      p += 4;                                                   \
+      if (!FETCH_DATA (info, p))                                \
+        return -3;                                              \
+      floatformat_to_double (& floatformat_ieee_single_big,     \
+                             (char *) p - 4, & val);            \
+    }                                                           \
+  while (0)
 
 /* Get a double precision float.  */
-#define NEXTDOUBLE(val, p) \
-  (p += 8, FETCH_DATA (info, p), \
-   floatformat_to_double (&floatformat_ieee_double_big, (char *) p - 8, &val))
+#define NEXTDOUBLE(val, p)                                      \
+  do                                                            \
+    {                                                           \
+      p += 8;                                                   \
+      if (!FETCH_DATA (info, p))                                \
+        return -3;                                              \
+      floatformat_to_double (& floatformat_ieee_double_big,     \
+                             (char *) p - 8, & val);            \
+    }                                                           \
+  while (0)
 
 /* Get an extended precision float.  */
-#define NEXTEXTEND(val, p) \
-  (p += 12, FETCH_DATA (info, p), \
-   floatformat_to_double (&floatformat_m68881_ext, (char *) p - 12, &val))
+#define NEXTEXTEND(val, p)                              \
+  do                                                    \
+    {                                                   \
+      p += 12;                                          \
+      if (!FETCH_DATA (info, p))                        \
+        return -3;                                      \
+      floatformat_to_double (& floatformat_m68881_ext,  \
+                             (char *) p - 12, & val);   \
+    }                                                   \
+  while (0)
 
 /* Need a function to convert from packed to double
    precision.   Actually, it's easier to print a
    packed number than a double anyway, so maybe
    there should be a special case to handle this... */
-#define NEXTPACKED(p) \
-  (p += 12, FETCH_DATA (info, p), 0.0)
+#define NEXTPACKED(p, val)                      \
+  do                                            \
+    {                                           \
+      p += 12;                                  \
+      if (!FETCH_DATA (info, p))                \
+        return -3;                              \
+      val = 0.0;                                \
+    }                                           \
+  while (0)
+
 
 
 /* Maximum length of an instruction.  */

opcodes/ppc-opc.c

@@ -1281 +1281 @@
 
   /* mfsprg can use 260..263 and 272..279.  mtsprg only uses spr 272..279
      If not BOOKE or 405, then both use only 272..275.  */
-  if (val <= 3
-      || (val < 0x10 && (insn & 0x100) != 0)
-      || (val - 0x10 > 3
-          && (dialect & (PPC_OPCODE_BOOKE | PPC_OPCODE_403)) == 0))
+  if ((val - 0x10 > 3 && (dialect & (PPC_OPCODE_BOOKE | PPC_OPCODE_405)) == 0)
+      || (val - 0x10 > 7 && (insn & 0x100) != 0)
+      || val <= 3
+      || (val & 8) != 0)
     *invalid = 1;
   return val & 7;
 }