Tim, Thanks. We will need to backport this change to the ACPICA source code. Bob
> -----Original Message----- > From: Tim Gardner [mailto:[email protected]] > Sent: Wednesday, December 05, 2012 9:39 AM > To: [email protected] > Cc: Tim Gardner; Len Brown; Rafael J. Wysocki; Lin Ming; Moore, Robert; > [email protected] > Subject: [PATCH 3.7-rc8] ACPICA: Fix ACPI mutex object allocation memory > leak on error > > An unattached mutex object is not released on error, thereby orphaning > memory. > > I stumbled on this when looking at the output of smatch: > > drivers/acpi/acpica/dsmethod.c:225 acpi_ds_begin_method_execution() error: > we previously assumed 'obj_desc->method.mutex' could be null (see line > 209) > > which led me to have a look at acpi_ds_create_method_mutex(). > > Cc: Len Brown <[email protected]> > Cc: "Rafael J. Wysocki" <[email protected]> > Cc: Lin Ming <[email protected]> > Cc: Bob Moore <[email protected]> > Cc: [email protected] > Cc: [email protected] #2.6.29+ > Signed-off-by: Tim Gardner <[email protected]> > --- > > This applied to the original version of dsmethod.c which was introduced in > 95b482a8d31116f3f5c2a5089569393234d06385 (2.6.29). > > drivers/acpi/acpica/dsmethod.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/acpi/acpica/dsmethod.c > b/drivers/acpi/acpica/dsmethod.c index aa9a5d4..fe89ea9 100644 > --- a/drivers/acpi/acpica/dsmethod.c > +++ b/drivers/acpi/acpica/dsmethod.c > @@ -151,6 +151,7 @@ acpi_ds_create_method_mutex(union acpi_operand_object > *method_desc) > > status = acpi_os_create_mutex(&mutex_desc->mutex.os_mutex); > if (ACPI_FAILURE(status)) { > + acpi_ut_delete_object_desc(mutex_desc); > return_ACPI_STATUS(status); > } > > -- > 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
