aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/char
diff options
context:
space:
mode:
authorKylene Jo Hall <kjhall@us.ibm.com>2006-04-22 05:36:35 -0400
committerLinus Torvalds <torvalds@g5.osdl.org>2006-04-22 12:19:53 -0400
commit59e89f3a091d5cf93f4b176aedcfded61ece5252 (patch)
tree927003d6297eb724ff220fb416aa357c25854221 /drivers/char
parentb9251b823b5e921c894eb135cb6c64abf483f50e (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/char')
-rw-r--r--drivers/char/tpm/tpm_bios.c2
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