elf: fix memcmp() overflow

name can be (and will be) smaller than section_name_start. Both name and
section_name_start are NULL-terminated so we can use strcmp.

Change-Id: Iec414810109f9267de20c25c0fe601df35daf89e
Signed-off-by: Benoît Ganne <bganne@cisco.com>

diff --git a/src/vppinfra/elf_clib.c b/src/vppinfra/elf_clib.c
index 7bb72ee..2bc05e1 100644 (file)
--- a/src/vppinfra/elf_clib.c
+++ b/src/vppinfra/elf_clib.c
@@ -153,7 +153,7 @@ clib_elf_parse_file (clib_elf_main_t * cem,
       clib_elf_section_bounds_t *b;
 
       /* Section name must begin with CLIB_ELF_SECTION key. */
-      if (memcmp (name, section_name_start, section_name_start_len))
+      if (strcmp ((char *) name, section_name_start))
        continue;
 
       name += section_name_start_len;