diff options
author | Kylene Jo Hall <kjhall@us.ibm.com> | 2006-04-22 05:36:35 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-04-22 12:19:53 -0400 |
commit | 59e89f3a091d5cf93f4b176aedcfded61ece5252 (patch) | |
tree | 927003d6297eb724ff220fb416aa357c25854221 /drivers | |
parent | b9251b823b5e921c894eb135cb6c64abf483f50e (diff) |
[PATCH] tpm: fix memory leak
The eventname was kmalloc'd and not freed in the *_show functions.
This bug was found by Coverity.
Signed-off-by: Kylene Hall <kjhall@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/char/tpm/tpm_bios.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/char/tpm/tpm_bios.c b/drivers/char/tpm/tpm_bios.c index 537aa45d8c67..0549e2a35df8 100644 --- a/drivers/char/tpm/tpm_bios.c +++ b/drivers/char/tpm/tpm_bios.c | |||
@@ -306,6 +306,7 @@ static int tpm_binary_bios_measurements_show(struct seq_file *m, void *v) | |||
306 | /* 5th: delimiter */ | 306 | /* 5th: delimiter */ |
307 | seq_putc(m, '\0'); | 307 | seq_putc(m, '\0'); |
308 | 308 | ||
309 | kfree(eventname); | ||
309 | return 0; | 310 | return 0; |
310 | } | 311 | } |
311 | 312 | ||
@@ -353,6 +354,7 @@ static int tpm_ascii_bios_measurements_show(struct seq_file *m, void *v) | |||
353 | /* 4th: eventname <= max + \'0' delimiter */ | 354 | /* 4th: eventname <= max + \'0' delimiter */ |
354 | seq_printf(m, " %s\n", eventname); | 355 | seq_printf(m, " %s\n", eventname); |
355 | 356 | ||
357 | kfree(eventname); | ||
356 | return 0; | 358 | return 0; |
357 | } | 359 | } |
358 | 360 | ||