diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2010-03-30 10:26:30 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-03-30 10:26:30 -0400 |
| commit | 246750ffa1b26335df4e485b4e4d28d83756ac43 (patch) | |
| tree | 8c28306f1b66bd89713c9dfe8738097d56056d06 | |
| parent | 4660d3d240ac6c92cd3ad33657ca302026bdc24b (diff) | |
| parent | 570b8fb505896e007fd3bb07573ba6640e51851d (diff) | |
Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/security-testing-2.6
* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/security-testing-2.6:
CRED: Fix memory leak in error handling
| -rw-r--r-- | kernel/cred.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/kernel/cred.c b/kernel/cred.c index 1ed8ca18790c..1b1129d0cce8 100644 --- a/kernel/cred.c +++ b/kernel/cred.c | |||
| @@ -364,7 +364,7 @@ struct cred *prepare_usermodehelper_creds(void) | |||
| 364 | 364 | ||
| 365 | new = kmem_cache_alloc(cred_jar, GFP_ATOMIC); | 365 | new = kmem_cache_alloc(cred_jar, GFP_ATOMIC); |
| 366 | if (!new) | 366 | if (!new) |
| 367 | return NULL; | 367 | goto free_tgcred; |
| 368 | 368 | ||
| 369 | kdebug("prepare_usermodehelper_creds() alloc %p", new); | 369 | kdebug("prepare_usermodehelper_creds() alloc %p", new); |
| 370 | 370 | ||
| @@ -397,6 +397,10 @@ struct cred *prepare_usermodehelper_creds(void) | |||
| 397 | 397 | ||
| 398 | error: | 398 | error: |
| 399 | put_cred(new); | 399 | put_cred(new); |
| 400 | free_tgcred: | ||
| 401 | #ifdef CONFIG_KEYS | ||
| 402 | kfree(tgcred); | ||
| 403 | #endif | ||
| 400 | return NULL; | 404 | return NULL; |
| 401 | } | 405 | } |
| 402 | 406 | ||
