[PATCH] lib: fwts_acpi_tables: unmap mem, fixes memory leak
Colin King
colin.king at canonical.com
Tue Aug 22 15:08:40 UTC 2017
From: Colin Ian King <colin.king at canonical.com>
I forgot to unmap mem before returning from the function, causing a
memory leak. Fix this by unmapping it.
Fixes: f327fcea6f81 ("lib: fwts_acpi_tables: re-work fwts_acpi_get_rsdp, less memory copies")
Signed-off-by: Colin Ian King <colin.king at canonical.com>
---
src/lib/src/fwts_acpi_tables.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/lib/src/fwts_acpi_tables.c b/src/lib/src/fwts_acpi_tables.c
index 9b2312e5..3c3a63a1 100644
--- a/src/lib/src/fwts_acpi_tables.c
+++ b/src/lib/src/fwts_acpi_tables.c
@@ -162,6 +162,7 @@ static fwts_acpi_table_rsdp *fwts_acpi_get_rsdp(fwts_framework *fw, void *addr,
fwts_log_error(fw, "Cannot safely copy RSDP from address %p.", mem);
goto err;
}
+ (void)fwts_munmap(mem, sizeof(fwts_acpi_table_rsdp));
/* Determine original RSDP size from revision. */
*rsdp_len = (rsdp->revision < 1) ? 20 : 36;
--
2.14.1
More information about the fwts-devel
mailing list