Commit c53d96a4 authored by Daniil Tatianin's avatar Daniil Tatianin Committed by Rafael J. Wysocki
Browse files

ACPICA: events/evxfregn: don't release the ContextMutex that was never acquired 

This bug was first introduced in c27f3d01, where the author of the
patch probably meant to do DeleteMutex instead of ReleaseMutex. The
mutex leak was noticed later on and fixed in e4dfe108, but the bogus
MutexRelease line was never removed, so do it now.

Link: https://github.com/acpica/acpica/pull/982


Fixes: c27f3d01 ("ACPICA: Fix race in generic_serial_bus (I2C) and GPIO op_region parameter handling")
Signed-off-by: default avatarDaniil Tatianin <d-tatianin@yandex-team.ru>
Link: https://patch.msgid.link/20241122082954.658356-1-d-tatianin@yandex-team.ru


Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent fac04efc

drivers/acpi/acpica/evxfregn.c

+0 −2
Original line number Diff line number Diff line
@@ -232,8 +232,6 @@ acpi_remove_address_space_handler(acpi_handle device, 


			/* Now we can delete the handler object */



			acpi_os_release_mutex(handler_obj->address_space.

					      context_mutex);

			acpi_ut_remove_reference(handler_obj);

			goto unlock_and_exit;

		}