diff options
author | Gustavo A. R. Silva <garsilva@embeddedor.com> | 2017-07-07 07:36:10 +0200 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2017-07-07 14:42:07 +0200 |
commit | e3b53b8a465133ce57f9722aab5617b8e0f9657f (patch) | |
tree | 7ca9388b45bbaf11f0313dc16ee28120c524888f /drivers/regulator | |
parent | regulator: qcom_rpm-regulator: add NULL check on of_match_device() return value (diff) | |
download | linux-e3b53b8a465133ce57f9722aab5617b8e0f9657f.tar.xz linux-e3b53b8a465133ce57f9722aab5617b8e0f9657f.zip |
regulator: qcom_smd: add NULL check on of_match_device() return value
Check return value from call to of_match_device()
in order to prevent a NULL pointer dereference.
In case of NULL print error message and return.
Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/regulator')
-rw-r--r-- | drivers/regulator/qcom_smd-regulator.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/regulator/qcom_smd-regulator.c b/drivers/regulator/qcom_smd-regulator.c index f35994a2a5be..940fe1b78411 100644 --- a/drivers/regulator/qcom_smd-regulator.c +++ b/drivers/regulator/qcom_smd-regulator.c @@ -570,6 +570,11 @@ static int rpm_reg_probe(struct platform_device *pdev) } match = of_match_device(rpm_of_match, &pdev->dev); + if (!match) { + dev_err(&pdev->dev, "failed to match device\n"); + return -ENODEV; + } + for (reg = match->data; reg->name; reg++) { vreg = devm_kzalloc(&pdev->dev, sizeof(*vreg), GFP_KERNEL); if (!vreg) |