summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKees Cook <keescook@chromium.org>2016-04-07 00:53:27 +0200
committerKees Cook <keescook@chromium.org>2016-04-07 01:22:25 +0200
commit3d085c7413d32bb6895e5b9b5ee6a7d2180159c5 (patch)
treeeb6ab087af58ec383ef8fc6286fa67b1185c00e6
parentlkdtm: fix memory leak of base (diff)
downloadlinux-3d085c7413d32bb6895e5b9b5ee6a7d2180159c5.tar.xz
linux-3d085c7413d32bb6895e5b9b5ee6a7d2180159c5.zip
lkdtm: do not leak free page on kmalloc failure
This frees the allocated page if there is a kmalloc failure. Signed-off-by: Kees Cook <keescook@chromium.org>
-rw-r--r--drivers/misc/lkdtm.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/misc/lkdtm.c b/drivers/misc/lkdtm.c
index 5b3a63c87ec9..0a5cbbe12452 100644
--- a/drivers/misc/lkdtm.c
+++ b/drivers/misc/lkdtm.c
@@ -507,8 +507,10 @@ static void lkdtm_do_action(enum ctype which)
break;
val = kmalloc(1024, GFP_KERNEL);
- if (!val)
+ if (!val) {
+ free_page(p);
break;
+ }
base = (int *)p;