diff options
author | LABBE Corentin <clabbe.montjoie@gmail.com> | 2016-08-16 05:27:18 -0400 |
---|---|---|
committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2016-09-27 04:41:30 -0400 |
commit | d8ed9e87e6016433b8d178728a5391ea68101bca (patch) | |
tree | 5b4a890f5466bbf9aa882c5191eeeb08115429e8 | |
parent | 60169c3fb78e5cd61cb1343e7f3c5edb72702a04 (diff) |
fbdev: ssd1307fb: fix a possible NULL dereference
of_match_device could return NULL, and so cause a NULL pointer
dereference later.
For fixing this problem, we use of_device_get_match_data(), this will
simplify the code a little by using a standard function for
getting the match data.
Signed-off-by: LABBE Corentin <clabbe.montjoie@gmail.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
-rw-r--r-- | drivers/video/fbdev/ssd1307fb.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/video/fbdev/ssd1307fb.c b/drivers/video/fbdev/ssd1307fb.c index 6a18bdc9fe47..2925d5ce8d3e 100644 --- a/drivers/video/fbdev/ssd1307fb.c +++ b/drivers/video/fbdev/ssd1307fb.c | |||
@@ -559,8 +559,7 @@ static int ssd1307fb_probe(struct i2c_client *client, | |||
559 | par->info = info; | 559 | par->info = info; |
560 | par->client = client; | 560 | par->client = client; |
561 | 561 | ||
562 | par->device_info = of_match_device(ssd1307fb_of_match, | 562 | par->device_info = of_device_get_match_data(&client->dev); |
563 | &client->dev)->data; | ||
564 | 563 | ||
565 | par->reset = of_get_named_gpio(client->dev.of_node, | 564 | par->reset = of_get_named_gpio(client->dev.of_node, |
566 | "reset-gpios", 0); | 565 | "reset-gpios", 0); |