[3/4] readelf: Fix off by one error whilst checking reloc location against section size.

Message ID 20191103075743.25467-4-ceggers@gmx.de
State New
Headers show
Series
  • readelf: Fixes for IN_RANGE
Related show

Commit Message

Christian Eggers Nov. 3, 2019, 7:57 a.m.
* readelf.c (IN_RANGE): Fix off by one error
	whilst checking reloc location against section size.

Signed-off-by: Christian Eggers <ceggers@gmx.de>

---
 binutils/ChangeLog | 2 ++
 binutils/readelf.c | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

--
2.16.4

Patch

diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index f25e6365ba..0e3e3fc535 100644
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -3,6 +3,8 @@ 
 	* readelf.c (IN_RANGE): Rename parameter OFF to SIZE.
 	* readelf.c (IN_RANGE): Catch potential integer overflow
 	whilst checking reloc location against section size.
+	* readelf.c (IN_RANGE): Fix off by one error
+	whilst checking reloc location against section size.

 2019-10-20  Palmer Dabbelt  <palmer@sifive.com>

diff --git a/binutils/readelf.c b/binutils/readelf.c
index 1854e866d7..f426cbb2be 100644
--- a/binutils/readelf.c
+++ b/binutils/readelf.c
@@ -12310,7 +12310,7 @@  process_syminfo (Filedata * filedata ATTRIBUTE_UNUSED)
 }

 #define IN_RANGE(START,END,ADDR,SIZE)		\
-  (((ADDR) >= (START)) && ((ADDR) < (END)) && ((ADDR) + (SIZE) < (END)))
+  (((ADDR) >= (START)) && ((ADDR) < (END)) && ((ADDR) + (SIZE) <= (END)))

 /* Check to see if the given reloc needs to be handled in a target specific
    manner.  If so then process the reloc and return TRUE otherwise return