From bcf83b2a66f0d968b51af8357f1543523ef83470 Mon Sep 17 00:00:00 2001 From: Nick Clifton Date: Thu, 20 Mar 2014 13:15:12 +0000 Subject: [PATCH] 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. --- binutils/ChangeLog | 5 +++++ binutils/readelf.c | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/binutils/ChangeLog b/binutils/ChangeLog index 03a1e18cda2..0f2707b44cd 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,8 @@ +2014-03-20 Nick Clifton + + * readelf.c (process_version_sections): Fix off-by-one error in + previous delta. + 2014-03-19 Nick Clifton PR binutils/16723 diff --git a/binutils/readelf.c b/binutils/readelf.c index 79137e3b3ed..c757a63ede6 100644 --- 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; -- 2.30.2