platform/x86: alienware-wmi-wmax: Fix NULL pointer dereference in sleep handlers
commit a49c4d48c3b60926e6a8cec217bf95aa65388ecc upstream.
Devices without the AWCC interface don't initialize `awcc`. Add a check
before dereferencing it in sleep handlers.
Cc: stable@vger.kernel.org
Reported-by: Gal Hammer <galhammer@gmail.com>
Tested-by: Gal Hammer <galhammer@gmail.com>
Fixes: 07ac275981 ("platform/x86: alienware-wmi-wmax: Add support for manual fan control")
Signed-off-by: Kurt Borja <kuurtb@gmail.com>
Link: https://patch.msgid.link/20251014-sleep-fix-v3-1-b5cb58da4638@gmail.com
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
9aac08036a
commit
24c3812c9e
@@ -1647,7 +1647,7 @@ static int wmax_wmi_probe(struct wmi_device *wdev, const void *context)
|
|||||||
|
|
||||||
static int wmax_wmi_suspend(struct device *dev)
|
static int wmax_wmi_suspend(struct device *dev)
|
||||||
{
|
{
|
||||||
if (awcc->hwmon)
|
if (awcc && awcc->hwmon)
|
||||||
awcc_hwmon_suspend(dev);
|
awcc_hwmon_suspend(dev);
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
@@ -1655,7 +1655,7 @@ static int wmax_wmi_suspend(struct device *dev)
|
|||||||
|
|
||||||
static int wmax_wmi_resume(struct device *dev)
|
static int wmax_wmi_resume(struct device *dev)
|
||||||
{
|
{
|
||||||
if (awcc->hwmon)
|
if (awcc && awcc->hwmon)
|
||||||
awcc_hwmon_resume(dev);
|
awcc_hwmon_resume(dev);
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
|
|||||||
Reference in New Issue
Block a user