Nick Clifton 0e219f
--- binutils.orig/bfd/dwarf1.c	2018-07-11 11:45:09.482030157 +0100
Nick Clifton 0e219f
+++ binutils-2.29/bfd/dwarf1.c	2018-07-11 11:48:19.564980423 +0100
Nick Clifton 0e219f
@@ -208,6 +208,7 @@ parse_die (bfd *             abfd,
Nick Clifton 0e219f
   /* Then the attributes.  */
Nick Clifton 0e219f
   while (xptr < (this_die + aDieInfo->length))
Nick Clifton 0e219f
     {
Nick Clifton 0e219f
+      unsigned int   block_len;
Nick Clifton 0e219f
       unsigned short attr;
Nick Clifton 0e219f
 
Nick Clifton 0e219f
       /* Parse the attribute based on its form.  This section
Nick Clifton 0e219f
@@ -243,10 +244,26 @@ parse_die (bfd *             abfd,
Nick Clifton 0e219f
 	  xptr += 4;
Nick Clifton 0e219f
 	  break;
Nick Clifton 0e219f
 	case FORM_BLOCK2:
Nick Clifton 0e219f
-	  xptr += 2 + bfd_get_16 (abfd, (bfd_byte *) xptr);
Nick Clifton 0e219f
+ 	  if (xptr + 2 <= aDiePtrEnd)
Nick Clifton 0e219f
+	    {
Nick Clifton 0e219f
+	      block_len = bfd_get_16 (abfd, xptr);
Nick Clifton 0e219f
+	      if (xptr + block_len > aDiePtrEnd
Nick Clifton 0e219f
+		  || xptr + block_len < xptr)
Nick Clifton 0e219f
+		return FALSE;
Nick Clifton 0e219f
+	      xptr += block_len;
Nick Clifton 0e219f
+	    }
Nick Clifton 0e219f
+ 	  xptr += 2;
Nick Clifton 0e219f
 	  break;
Nick Clifton 0e219f
 	case FORM_BLOCK4:
Nick Clifton 0e219f
-	  xptr += 4 + bfd_get_32 (abfd, (bfd_byte *) xptr);
Nick Clifton 0e219f
+	  if (xptr + 4 <= aDiePtrEnd)
Nick Clifton 0e219f
+	    {
Nick Clifton 0e219f
+	      block_len = bfd_get_32 (abfd, xptr);
Nick Clifton 0e219f
+	      if (xptr + block_len > aDiePtrEnd
Nick Clifton 0e219f
+		  || xptr + block_len < xptr)
Nick Clifton 0e219f
+		return FALSE;
Nick Clifton 0e219f
+	      xptr += block_len;
Nick Clifton 0e219f
+	    }
Nick Clifton 0e219f
+ 	  xptr += 4;
Nick Clifton 0e219f
 	  break;
Nick Clifton 0e219f
 	case FORM_STRING:
Nick Clifton 0e219f
 	  if (attr == AT_name)