diff options
author | Vinod Koul <vkoul@kernel.org> | 2018-08-07 08:24:50 +0200 |
---|---|---|
committer | Vinod Koul <vkoul@kernel.org> | 2018-08-27 06:19:48 +0200 |
commit | e1c815f4b24a305e5bc9eceb541674bf4d02b709 (patch) | |
tree | cc809a843249155a91f7af37b19b182feebcc4aa /drivers | |
parent | soundwire: intel: Add pre/post bank switch ops (diff) | |
download | linux-e1c815f4b24a305e5bc9eceb541674bf4d02b709.tar.xz linux-e1c815f4b24a305e5bc9eceb541674bf4d02b709.zip |
soundwire: intel: Fix uninitialized adev deref
In case of error, we can dereference uninitialized 'adev'
drivers/soundwire/intel_init.c:154 sdw_intel_acpi_cb()
error: uninitialized symbol 'adev'.
Fix that by not using adev for warn print and make it pr_err.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/soundwire/intel_init.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/soundwire/intel_init.c b/drivers/soundwire/intel_init.c index d1ea6b4d0ad3..5c8a20d99878 100644 --- a/drivers/soundwire/intel_init.c +++ b/drivers/soundwire/intel_init.c @@ -151,7 +151,7 @@ static acpi_status sdw_intel_acpi_cb(acpi_handle handle, u32 level, struct acpi_device *adev; if (acpi_bus_get_device(handle, &adev)) { - dev_err(&adev->dev, "Couldn't find ACPI handle\n"); + pr_err("%s: Couldn't find ACPI handle\n", __func__); return AE_NOT_FOUND; } |