[PATCH] nfp: Fix a potential out-of-bound access

Yinjun Zhang yinjun.zhang@corigine.com
Fri Sep 3 01:43:55 GMT 2021


Add necessary sanity check.

Reported-by: Alan Modra <amodra@gmail.com>
Signed-off-by: Yinjun Zhang <yinjun.zhang@corigine.com>
Signed-off-by: Simon Horman <simon.horman@corigine.com>
---
 opcodes/nfp-dis.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/opcodes/nfp-dis.c b/opcodes/nfp-dis.c
index 170f6a2d63e..88db34cac10 100644
--- a/opcodes/nfp-dis.c
+++ b/opcodes/nfp-dis.c
@@ -2644,6 +2644,9 @@ init_nfp6000_mecsr_sec (nfp_priv_data * priv, Elf_Internal_Shdr * sec,
       menum = _BF (ireg.cpp_offset_lo, 13, 10) - 4;
       csr_off = _BF (ireg.cpp_offset_lo, 9, 0);
 
+      if (menum >= _NFP_ME_MAX)
+	return false;
+
       mecfg = &priv->mecfgs[isl][menum][is_for_text];
       switch (csr_off)
 	{
-- 
2.27.0



More information about the Binutils mailing list