lkml.org 
[lkml]   [2021]   [Oct]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH] module: fix elf_validity_check() warns seen on 32-bit platforms
Date
Fix the following warnings introduced by

commit: 8b1185a4427b ("module: change to print useful messages from elf_validity_check()")

warning: format '%llu' expects argument of type 'long long unsigned int',
but argument 3 has type 'Elf32_Off' {aka 'unsigned int'}

Fix it by tweaking messages to not print ELF64* fields.

Fixes: 8b1185a4427b ("module: change to print useful messages from elf_validity_check()")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
---
kernel/module.c | 19 +++++++------------
1 file changed, 7 insertions(+), 12 deletions(-)

diff --git a/kernel/module.c b/kernel/module.c
index d972786b10ba..ebc70239f65a 100644
--- a/kernel/module.c
+++ b/kernel/module.c
@@ -2950,8 +2950,8 @@ static int validate_section_offset(struct load_info *info, Elf_Shdr *shdr)
*/
secend = shdr->sh_offset + shdr->sh_size;
if (secend < shdr->sh_offset || secend > info->len) {
- pr_err("Invalid ELF section offset/size: secend(%lu) < shdr->sh_offset(%llu) or secend(%lu) > e_shnum(%lu)\n",
- secend, shdr->sh_offset, secend, info->len);
+ pr_err("Invalid ELF section offset/size: sh_offset+sh_size(%lu) < sh_offset || > len(%lu)\n",
+ secend, info->len);
return -ENOEXEC;
}

@@ -2971,8 +2971,7 @@ static int elf_validity_check(struct load_info *info)
int err;

if (info->len < sizeof(*(info->hdr))) {
- pr_err("Invalid ELF header len %lu < %lu\n", info->len,
- sizeof(*(info->hdr)));
+ pr_err("Invalid ELF header len %lu\n", info->len);
goto no_exec;
}

@@ -2991,8 +2990,7 @@ static int elf_validity_check(struct load_info *info)
goto no_exec;
}
if (info->hdr->e_shentsize != sizeof(Elf_Shdr)) {
- pr_err("Invalid ELF section header size %d != %lu\n",
- info->hdr->e_shentsize, sizeof(Elf_Shdr));
+ pr_err("Invalid ELF section header size\n");
goto no_exec;
}

@@ -3004,9 +3002,7 @@ static int elf_validity_check(struct load_info *info)
if (info->hdr->e_shoff >= info->len
|| (info->hdr->e_shnum * sizeof(Elf_Shdr) >
info->len - info->hdr->e_shoff)) {
- pr_err("Invalid ELF section header overflow: %ld > %llu\n",
- info->hdr->e_shnum * sizeof(Elf_Shdr),
- info->len - info->hdr->e_shoff);
+ pr_err("Invalid ELF section header overflow\n");
goto no_exec;
}

@@ -3046,9 +3042,8 @@ static int elf_validity_check(struct load_info *info)
if (info->sechdrs[0].sh_type != SHT_NULL
|| info->sechdrs[0].sh_size != 0
|| info->sechdrs[0].sh_addr != 0) {
- pr_err("ELF Spec violation: section 0 type!=SH_NULL(%d) or non-zero len(%llu) or addr(%llu)\n",
- info->sechdrs[0].sh_type, info->sechdrs[0].sh_size,
- info->sechdrs[0].sh_addr);
+ pr_err("ELF Spec violation: section 0 type(%d)!=SH_NULL or non-zero len or addr\n",
+ info->sechdrs[0].sh_type);
goto no_exec;
}

--
2.30.2
\
 
 \ /
  Last update: 2021-10-14 20:11    [W:0.058 / U:0.972 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site