module_total_size() returns unsigned int but uses a signed int
accumulator. While the result is numerically correct, the type
mismatch is misleading.

Change the accumulator to unsigned int to match the return type.

Signed-off-by: Naveen Kumar Chaudhary <[email protected]>
---
Changes in v2:
    - Modify the commit message

Thanks Sami for the catch. I have corrected the commit message.

Regards,
Naveen

 kernel/module/procfs.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/module/procfs.c b/kernel/module/procfs.c
index 0a4841e88adb..90712aa9dd13 100644
--- a/kernel/module/procfs.c
+++ b/kernel/module/procfs.c
@@ -64,7 +64,7 @@ static void m_stop(struct seq_file *m, void *p)
 
 static unsigned int module_total_size(struct module *mod)
 {
-       int size = 0;
+       unsigned int size = 0;
 
        for_each_mod_mem_type(type)
                size += mod->mem[type].size;
-- 
2.43.0


Reply via email to