diff options
| author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2015-04-14 12:53:36 +0200 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2015-04-14 12:53:36 +0200 |
| commit | 03aecbcd71f2d9b9a3bf422a880a1596fd585880 (patch) | |
| tree | 56b39978f85621840c1827903c3f578445d0da8e | |
| parent | Merge branches 'pm-sleep' and 'pm-domains' (diff) | |
| parent | ACPI / scan: Fix NULL pointer dereference in acpi_companion_match() (diff) | |
| download | linux-03aecbcd71f2d9b9a3bf422a880a1596fd585880.tar.xz linux-03aecbcd71f2d9b9a3bf422a880a1596fd585880.zip | |
Merge branch 'acpi-scan'
* acpi-scan:
ACPI / scan: Fix NULL pointer dereference in acpi_companion_match()
| -rw-r--r-- | drivers/acpi/scan.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c index d2e3c3e3f9c9..69bc0d888c01 100644 --- a/drivers/acpi/scan.c +++ b/drivers/acpi/scan.c @@ -247,6 +247,7 @@ static int create_of_modalias(struct acpi_device *acpi_dev, char *modalias, static struct acpi_device *acpi_companion_match(const struct device *dev) { struct acpi_device *adev; + struct mutex *physical_node_lock; adev = ACPI_COMPANION(dev); if (!adev) @@ -255,7 +256,8 @@ static struct acpi_device *acpi_companion_match(const struct device *dev) if (list_empty(&adev->pnp.ids)) return NULL; - mutex_lock(&adev->physical_node_lock); + physical_node_lock = &adev->physical_node_lock; + mutex_lock(physical_node_lock); if (list_empty(&adev->physical_node_list)) { adev = NULL; } else { @@ -266,7 +268,7 @@ static struct acpi_device *acpi_companion_match(const struct device *dev) if (node->dev != dev) adev = NULL; } - mutex_unlock(&adev->physical_node_lock); + mutex_unlock(physical_node_lock); return adev; } |