[PATCH] seabios: acpi: fix memory leak in build_srat().

numadata() is allocated for temporal use, but not freed. free it. Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> --- src/acpi.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/src/acpi.c b/src/acpi.c index dbdca79..e2a1153 100644 --- a/src/acpi.c +++ b/src/acpi.c @@ -490,6 +490,7 @@ build_srat(void) srat = malloc_high(srat_size); if (!srat) { dprintf(1, "Not enough memory for srat table!\n"); + free(numadata); return NULL; } @@ -556,6 +557,7 @@ build_srat(void) build_header((void*)srat, SRAT_SIGNATURE, srat_size, 1); + free(numadata); return srat; } -- 1.6.6.1

On Fri, Feb 12, 2010 at 11:36:20AM +0900, Isaku Yamahata wrote:
numadata() is allocated for temporal use, but not freed. free it.
Thanks. -Kevin
participants (2)
-
Isaku Yamahata
-
Kevin O'Connor