Commit 6fe2e70b authored by Jayachandran C's avatar Jayachandran C Committed by Linus Torvalds

[PATCH] kernel/module.c: removed dead code

This patch fixes an issue reported by Coverity in kernel/module.c

Error reported: Cannot reach this line of code "else return ptr;"

Patch description:
  This is the error path, so 'err' will be negative, the else case
  is not required, this patch removes it.
Signed-off-by: default avatarJayachandran C. <c.jayachandran@gmail.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 066bb8d0
...@@ -1854,8 +1854,7 @@ static struct module *load_module(void __user *umod, ...@@ -1854,8 +1854,7 @@ static struct module *load_module(void __user *umod,
kfree(args); kfree(args);
free_hdr: free_hdr:
vfree(hdr); vfree(hdr);
if (err < 0) return ERR_PTR(err); return ERR_PTR(err);
else return ptr;
truncated: truncated:
printk(KERN_ERR "Module len %lu truncated\n", len); printk(KERN_ERR "Module len %lu truncated\n", len);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment