Nick Clifton 9422ba
--- binutils.orig/bfd/dwarf2.c	2018-07-11 11:45:09.491030059 +0100
Nick Clifton 9422ba
+++ binutils-2.29/bfd/dwarf2.c	2018-07-11 12:16:12.975943400 +0100
Nick Clifton 9422ba
@@ -626,14 +626,24 @@ read_8_bytes (bfd *abfd, bfd_byte *buf,
Nick Clifton 9422ba
 }
Nick Clifton 9422ba
 
Nick Clifton 9422ba
 static bfd_byte *
Nick Clifton 9422ba
-read_n_bytes (bfd *abfd ATTRIBUTE_UNUSED,
Nick Clifton 9422ba
-	      bfd_byte *buf,
Nick Clifton 9422ba
-	      bfd_byte *end,
Nick Clifton 9422ba
-	      unsigned int size ATTRIBUTE_UNUSED)
Nick Clifton 9422ba
-{
Nick Clifton 9422ba
-  if (buf + size > end)
Nick Clifton 9422ba
-    return NULL;
Nick Clifton 9422ba
-  return buf;
Nick Clifton 9422ba
+read_n_bytes (bfd_byte *           buf,
Nick Clifton 9422ba
+	      bfd_byte *           end,
Nick Clifton 9422ba
+	      struct dwarf_block * block)
Nick Clifton 9422ba
+{
Nick Clifton 9422ba
+  unsigned int  size = block->size;
Nick Clifton 9422ba
+  bfd_byte *    block_end = buf + size;
Nick Clifton 9422ba
+
Nick Clifton 9422ba
+  if (block_end > end || block_end < buf)
Nick Clifton 9422ba
+    {
Nick Clifton 9422ba
+      block->data = NULL;
Nick Clifton 9422ba
+      block->size = 0;
Nick Clifton 9422ba
+      return end;
Nick Clifton 9422ba
+    }
Nick Clifton 9422ba
+  else
Nick Clifton 9422ba
+    {
Nick Clifton 9422ba
+      block->data = buf;
Nick Clifton 9422ba
+      return block_end;
Nick Clifton 9422ba
+    }
Nick Clifton 9422ba
 }
Nick Clifton 9422ba
 
Nick Clifton 9422ba
 /* Scans a NUL terminated string starting at BUF, returning a pointer to it.
Nick Clifton 9422ba
@@ -1131,8 +1141,7 @@ read_attribute_value (struct attribute *
Nick Clifton 9422ba
 	return NULL;
Nick Clifton 9422ba
       blk->size = read_2_bytes (abfd, info_ptr, info_ptr_end);
Nick Clifton 9422ba
       info_ptr += 2;
Nick Clifton 9422ba
-      blk->data = read_n_bytes (abfd, info_ptr, info_ptr_end, blk->size);
Nick Clifton 9422ba
-      info_ptr += blk->size;
Nick Clifton 9422ba
+      info_ptr = read_n_bytes (info_ptr, info_ptr_end, blk);
Nick Clifton 9422ba
       attr->u.blk = blk;
Nick Clifton 9422ba
       break;
Nick Clifton 9422ba
     case DW_FORM_block4:
Nick Clifton 9422ba
@@ -1142,8 +1151,7 @@ read_attribute_value (struct attribute *
Nick Clifton 9422ba
 	return NULL;
Nick Clifton 9422ba
       blk->size = read_4_bytes (abfd, info_ptr, info_ptr_end);
Nick Clifton 9422ba
       info_ptr += 4;
Nick Clifton 9422ba
-      blk->data = read_n_bytes (abfd, info_ptr, info_ptr_end, blk->size);
Nick Clifton 9422ba
-      info_ptr += blk->size;
Nick Clifton 9422ba
+      info_ptr = read_n_bytes (info_ptr, info_ptr_end, blk);
Nick Clifton 9422ba
       attr->u.blk = blk;
Nick Clifton 9422ba
       break;
Nick Clifton 9422ba
     case DW_FORM_data2:
Nick Clifton 9422ba
@@ -1183,8 +1191,7 @@ read_attribute_value (struct attribute *
Nick Clifton 9422ba
       blk->size = _bfd_safe_read_leb128 (abfd, info_ptr, &bytes_read,
Nick Clifton 9422ba
 					 FALSE, info_ptr_end);
Nick Clifton 9422ba
       info_ptr += bytes_read;
Nick Clifton 9422ba
-      blk->data = read_n_bytes (abfd, info_ptr, info_ptr_end, blk->size);
Nick Clifton 9422ba
-      info_ptr += blk->size;
Nick Clifton 9422ba
+      info_ptr = read_n_bytes (info_ptr, info_ptr_end, blk);
Nick Clifton 9422ba
       attr->u.blk = blk;
Nick Clifton 9422ba
       break;
Nick Clifton 9422ba
     case DW_FORM_block1:
Nick Clifton 9422ba
@@ -1194,8 +1201,7 @@ read_attribute_value (struct attribute *
Nick Clifton 9422ba
 	return NULL;
Nick Clifton 9422ba
       blk->size = read_1_byte (abfd, info_ptr, info_ptr_end);
Nick Clifton 9422ba
       info_ptr += 1;
Nick Clifton 9422ba
-      blk->data = read_n_bytes (abfd, info_ptr, info_ptr_end, blk->size);
Nick Clifton 9422ba
-      info_ptr += blk->size;
Nick Clifton 9422ba
+      info_ptr = read_n_bytes (info_ptr, info_ptr_end, blk);
Nick Clifton 9422ba
       attr->u.blk = blk;
Nick Clifton 9422ba
       break;
Nick Clifton 9422ba
     case DW_FORM_data1: