Nick Clifton 25d3f6
--- binutils.orig/bfd/opncls.c	2018-07-11 14:28:44.690234489 +0100
Nick Clifton 25d3f6
+++ binutils-2.29.1/bfd/opncls.c	2018-07-11 15:14:54.576580916 +0100
Nick Clifton 25d3f6
@@ -1179,6 +1179,7 @@ bfd_get_debug_link_info_1 (bfd *abfd, vo
Nick Clifton 25d3f6
   bfd_byte *contents;
Nick Clifton 25d3f6
   unsigned int crc_offset;
Nick Clifton 25d3f6
   char *name;
Nick Clifton 25d3f6
+  bfd_size_type size;
Nick Clifton 25d3f6
 
Nick Clifton 25d3f6
   BFD_ASSERT (abfd);
Nick Clifton 25d3f6
   BFD_ASSERT (crc32_out);
Nick Clifton 25d3f6
@@ -1188,6 +1189,12 @@ bfd_get_debug_link_info_1 (bfd *abfd, vo
Nick Clifton 25d3f6
   if (sect == NULL)
Nick Clifton 25d3f6
     return NULL;
Nick Clifton 25d3f6
 
Nick Clifton 25d3f6
+  size = bfd_get_section_size (sect);
Nick Clifton 25d3f6
+
Nick Clifton 25d3f6
+  /* PR 22794: Make sure that the section has a reasonable size.  */
Nick Clifton 25d3f6
+  if (size < 8 || size >= bfd_get_size (abfd))
Nick Clifton 25d3f6
+    return NULL;
Nick Clifton 25d3f6
+
Nick Clifton 25d3f6
   if (!bfd_malloc_and_get_section (abfd, sect, &contents))
Nick Clifton 25d3f6
     {
Nick Clifton 25d3f6
       if (contents != NULL)
Nick Clifton 25d3f6
@@ -1198,9 +1205,9 @@ bfd_get_debug_link_info_1 (bfd *abfd, vo
Nick Clifton 25d3f6
   /* CRC value is stored after the filename, aligned up to 4 bytes.  */
Nick Clifton 25d3f6
   name = (char *) contents;
Nick Clifton 25d3f6
   /* PR 17597: avoid reading off the end of the buffer.  */
Nick Clifton 25d3f6
-  crc_offset = strnlen (name, bfd_get_section_size (sect)) + 1;
Nick Clifton 25d3f6
+  crc_offset = strnlen (name, size) + 1;
Nick Clifton 25d3f6
   crc_offset = (crc_offset + 3) & ~3;
Nick Clifton 25d3f6
-  if (crc_offset >= bfd_get_section_size (sect))
Nick Clifton 25d3f6
+  if (crc_offset >= size)
Nick Clifton 25d3f6
     return NULL;
Nick Clifton 25d3f6
 
Nick Clifton 25d3f6
   *crc32 = bfd_get_32 (abfd, contents + crc_offset);
Nick Clifton 25d3f6
@@ -1261,6 +1268,7 @@ bfd_get_alt_debug_link_info (bfd * abfd,
Nick Clifton 25d3f6
   bfd_byte *contents;
Nick Clifton 25d3f6
   unsigned int buildid_offset;
Nick Clifton 25d3f6
   char *name;
Nick Clifton 25d3f6
+  bfd_size_type size;
Nick Clifton 25d3f6
 
Nick Clifton 25d3f6
   BFD_ASSERT (abfd);
Nick Clifton 25d3f6
   BFD_ASSERT (buildid_len);
Nick Clifton 25d3f6
@@ -1271,6 +1279,10 @@ bfd_get_alt_debug_link_info (bfd * abfd,
Nick Clifton 25d3f6
   if (sect == NULL)
Nick Clifton 25d3f6
     return NULL;
Nick Clifton 25d3f6
 
Nick Clifton 25d3f6
+  size = bfd_get_section_size (sect);
Nick Clifton 25d3f6
+  if (size < 8 || size >= bfd_get_size (abfd))
Nick Clifton 25d3f6
+    return NULL;
Nick Clifton 25d3f6
+
Nick Clifton 25d3f6
   if (!bfd_malloc_and_get_section (abfd, sect, & contents))
Nick Clifton 25d3f6
     {
Nick Clifton 25d3f6
       if (contents != NULL)
Nick Clifton 25d3f6
@@ -1280,11 +1292,11 @@ bfd_get_alt_debug_link_info (bfd * abfd,
Nick Clifton 25d3f6
 
Nick Clifton 25d3f6
   /* BuildID value is stored after the filename.  */
Nick Clifton 25d3f6
   name = (char *) contents;
Nick Clifton 25d3f6
-  buildid_offset = strnlen (name, bfd_get_section_size (sect)) + 1;
Nick Clifton 25d3f6
+  buildid_offset = strnlen (name, size) + 1;
Nick Clifton 25d3f6
   if (buildid_offset >= bfd_get_section_size (sect))
Nick Clifton 25d3f6
     return NULL;
Nick Clifton 25d3f6
 
Nick Clifton 25d3f6
-  *buildid_len = bfd_get_section_size (sect) - buildid_offset;
Nick Clifton 25d3f6
+  *buildid_len = size - buildid_offset;
Nick Clifton 25d3f6
   *buildid_out = bfd_malloc (*buildid_len);
Nick Clifton 25d3f6
   memcpy (*buildid_out, contents + buildid_offset, *buildid_len);
Nick Clifton 25d3f6