diff options
| author |   Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2016-06-06 17:25:33 +0300 |
|---|---|---|
| committer |   Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2016-07-21 22:55:53 +0200 |
| commit | 6ec39cf5cd6f77e3ff9ff1da75b758a47f939888 (patch) | |
| tree | 710f72f4368e76730c415432b2c1640a5363d593 /drivers/pci/pci.c | |
| parent | Linux 4.7-rc7 (diff) | |
| download | linux-dev-6ec39cf5cd6f77e3ff9ff1da75b758a47f939888.tar.xz linux-dev-6ec39cf5cd6f77e3ff9ff1da75b758a47f939888.zip | |
PCI / PM: check all fields in pci_set_platform_pm()
When assign new PCI platform PM operations check for all mandatory fields to
prevent NULL pointer dereference.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/pci/pci.c')
| -rw-r--r-- | drivers/pci/pci.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c index c8b4dbdd1bdd..badbddc683f0 100644 --- a/drivers/pci/pci.c +++ b/drivers/pci/pci.c @@ -530,8 +530,8 @@ static const struct pci_platform_pm_ops *pci_platform_pm; int pci_set_platform_pm(const struct pci_platform_pm_ops *ops) { - if (!ops->is_manageable || !ops->set_state || !ops->choose_state - || !ops->sleep_wake) + if (!ops->is_manageable || !ops->set_state || !ops->choose_state || + !ops->sleep_wake || !ops->run_wake || !ops->need_resume) return -EINVAL; pci_platform_pm = ops; return 0; |