summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJayachandran C <c.jayachandran@gmail.com>2006-01-06 00:19:54 -0800
committerLinus Torvalds <torvalds@g5.osdl.org>2006-01-06 08:33:59 -0800
commit6fe2e70bbed3995d930f39452fb6ce3be7dc47dc (patch)
treeb196b26fa0046c39bf072238463329dc66ccca82
parent066bb8d03b6e52e4844d37145573d6a2bedaa339 (diff)
downloadop-kernel-dev-6fe2e70bbed3995d930f39452fb6ce3be7dc47dc.zip
op-kernel-dev-6fe2e70bbed3995d930f39452fb6ce3be7dc47dc.tar.gz
[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: Jayachandran C. <c.jayachandran@gmail.com> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r--kernel/module.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/kernel/module.c b/kernel/module.c
index 2ea929d..4b06bba 100644
--- a/kernel/module.c
+++ b/kernel/module.c
@@ -1854,8 +1854,7 @@ static struct module *load_module(void __user *umod,
kfree(args);
free_hdr:
vfree(hdr);
- if (err < 0) return ERR_PTR(err);
- else return ptr;
+ return ERR_PTR(err);
truncated:
printk(KERN_ERR "Module len %lu truncated\n", len);
OpenPOWER on IntegriCloud