diff options
author | Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp> | 2011-01-17 03:22:47 +0300 |
---|---|---|
committer | James Morris <jmorris@namei.org> | 2011-01-19 01:53:53 +0300 |
commit | 35576eab390df313095306e2a8216134910e7014 (patch) | |
tree | c35b52f6797ce69091c3e3bc596783f45e19496a /security/apparmor | |
parent | 40c1001792de63e0f90e977eb05393fd71f78692 (diff) | |
download | linux-35576eab390df313095306e2a8216134910e7014.tar.xz |
trusted-keys: another free memory bugfix
TSS_rawhmac() forgot to call va_end()/kfree() when data == NULL and
forgot to call va_end() when crypto_shash_update() < 0.
Fix these bugs by escaping from the loop using "break"
(rather than "return"/"goto") in order to make sure that
va_end()/kfree() are always called.
Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Reviewed-by: Jesper Juhl <jj@chaosbits.net>
Acked-by: Mimi Zohar <zohar@us.ibm.com>
Acked-by: David Howells <dhowells@redhat.com>
Signed-off-by: James Morris <jmorris@namei.org>
Diffstat (limited to 'security/apparmor')
0 files changed, 0 insertions, 0 deletions