diff options
author | Kees Cook <keescook@chromium.org> | 2016-04-06 18:53:27 -0400 |
---|---|---|
committer | Kees Cook <keescook@chromium.org> | 2016-04-06 19:22:25 -0400 |
commit | 3d085c7413d32bb6895e5b9b5ee6a7d2180159c5 (patch) | |
tree | eb6ab087af58ec383ef8fc6286fa67b1185c00e6 | |
parent | d2e10088ce01191748857296999b8c24a9a51959 (diff) |
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.c | 4 |
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) | |||
507 | break; | 507 | break; |
508 | 508 | ||
509 | val = kmalloc(1024, GFP_KERNEL); | 509 | val = kmalloc(1024, GFP_KERNEL); |
510 | if (!val) | 510 | if (!val) { |
511 | free_page(p); | ||
511 | break; | 512 | break; |
513 | } | ||
512 | 514 | ||
513 | base = (int *)p; | 515 | base = (int *)p; |
514 | 516 | ||