An off-by-one error in the code to catch bogus vn_next fields meant that

linker testsuite failures were showing up for the cris target.  Fixed by
this patch.

	* readelf.c (process_version_sections): Fix off-by-one error in
	previous delta.

diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index 03a1e18..0f2707b 100644 (file)
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -1,3 +1,8 @@
+2014-03-20  Nick Clifton  <nickc@redhat.com>
+
+       * readelf.c (process_version_sections): Fix off-by-one error in
+       previous delta.
+
 2014-03-19  Nick Clifton  <nickc@redhat.com>
 
        PR binutils/16723

diff --git a/binutils/readelf.c b/binutils/readelf.c
index 79137e3..c757a63 100644 (file)
--- a/binutils/readelf.c
+++ b/binutils/readelf.c
@@ -8971,7 +8971,7 @@ process_version_sections (FILE * file)
                if (j < ent.vn_cnt)
                  warn (_("Missing Version Needs auxillary information\n"));
 
-               if (ent.vn_next == 0 && cnt < section->sh_info)
+               if (ent.vn_next == 0 && cnt < section->sh_info - 1)
                  {
                    warn (_("Corrupt Version Needs structure - offset to next structure is zero with entries still left to be processed\n"));
                    cnt = section->sh_info;