- memory-leak-in-acpi_evaluate_integer.patch removed from -mm tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The patch titled

     acpi: memory leak in acpi_evaluate_integer()

has been removed from the -mm tree.  Its filename is

     memory-leak-in-acpi_evaluate_integer.patch

This patch was probably dropped from -mm because
it has now been merged into a subsystem tree or
into Linus's tree, or because it was folded into
its parent patch in the -mm tree.


From: Vasily Averin <vvs@xxxxx>

acpi_evaluate_integer() does not release allocated memory on the error path.

(akpm: this is what happens when one uses multiple `return' statements...)

Signed-off-by: Vasily Averin <vvs@xxxxx>
Cc: "Brown, Len" <len.brown@xxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxx>
---

 drivers/acpi/utils.c |    2 ++
 1 files changed, 2 insertions(+)

diff -puN drivers/acpi/utils.c~memory-leak-in-acpi_evaluate_integer drivers/acpi/utils.c
--- devel/drivers/acpi/utils.c~memory-leak-in-acpi_evaluate_integer	2006-05-11 14:36:02.000000000 -0700
+++ devel-akpm/drivers/acpi/utils.c	2006-05-11 14:36:31.000000000 -0700
@@ -270,11 +270,13 @@ acpi_evaluate_integer(acpi_handle handle
 	status = acpi_evaluate_object(handle, pathname, arguments, &buffer);
 	if (ACPI_FAILURE(status)) {
 		acpi_util_eval_error(handle, pathname, status);
+		kfree(element);
 		return status;
 	}
 
 	if (element->type != ACPI_TYPE_INTEGER) {
 		acpi_util_eval_error(handle, pathname, AE_BAD_DATA);
+		kfree(element);
 		return AE_BAD_DATA;
 	}
 
_

Patches currently in -mm which might be from vvs@xxxxx are

git-acpi.patch

-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Kernel Newbies FAQ]     [Kernel Archive]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Photo]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]

  Powered by Linux