diff options
| author | Gustavo A. R. Silva <garsilva@embeddedor.com> | 2017-08-02 04:20:53 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-08-10 23:27:55 +0200 |
| commit | 26ffca5ec406213da347cd3c5858d1414943caea (patch) | |
| tree | 2774e8898d0808f219b6627f6243507111e29e72 /drivers/fpga | |
| parent | ARM: socfpga: explicitly request exclusive reset control (diff) | |
| download | linux-26ffca5ec406213da347cd3c5858d1414943caea.tar.xz linux-26ffca5ec406213da347cd3c5858d1414943caea.zip | |
fpga: altera-hps2fpga: 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 -ENODEV
Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Reviewed-by: Moritz Fischer <mdf@kernel.org>
Signed-off-by: Alan Tull <atull@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/fpga')
| -rw-r--r-- | drivers/fpga/altera-hps2fpga.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/fpga/altera-hps2fpga.c b/drivers/fpga/altera-hps2fpga.c index a18ff430c9af..e6c948b31474 100644 --- a/drivers/fpga/altera-hps2fpga.c +++ b/drivers/fpga/altera-hps2fpga.c @@ -143,6 +143,11 @@ static int alt_fpga_bridge_probe(struct platform_device *pdev) int ret; of_id = of_match_device(altera_fpga_of_match, dev); + if (!of_id) { + dev_err(dev, "failed to match device\n"); + return -ENODEV; + } + priv = (struct altera_hps2fpga_data *)of_id->data; priv->bridge_reset = of_reset_control_get_exclusive_by_index(dev->of_node, |