diff options
author | Gustavo A. R. Silva <garsilva@embeddedor.com> | 2017-07-07 00:29:58 -0500 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2017-07-07 13:42:03 +0100 |
commit | 2f2472baa66ef9a6aa25124cb07679bbe613ceff (patch) | |
tree | 93518a72aee34f642dc06656bd523444b228d2da /drivers/regulator | |
parent | 8d67f64f7739464c352d4b167ed1748f3b2c1f44 (diff) | |
download | op-kernel-dev-2f2472baa66ef9a6aa25124cb07679bbe613ceff.zip op-kernel-dev-2f2472baa66ef9a6aa25124cb07679bbe613ceff.tar.gz |
regulator: qcom_rpm-regulator: 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_rpm-regulator.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/regulator/qcom_rpm-regulator.c b/drivers/regulator/qcom_rpm-regulator.c index 1b2acc4..88dc0b0 100644 --- a/drivers/regulator/qcom_rpm-regulator.c +++ b/drivers/regulator/qcom_rpm-regulator.c @@ -959,6 +959,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_kmalloc(&pdev->dev, sizeof(*vreg), GFP_KERNEL); if (!vreg) |