[LEDE-DEV] [PATCH] ar71xx: Fix NULL pointer dereferece in at803x_link_change_notify()
Andrey Jr. Melnikov
temnota.am at gmail.com
Sat Oct 7 14:00:28 PDT 2017
Check pdata pointer before use to avoid OOPS if dev_get_platdata() return
NULL.
Signed-off-by: "Andrey Jr. Melnikov" <temnota.am at gmail.com>
---
diff --git a/target/linux/ar71xx/patches-4.4/902-at803x-add-reset-gpio-pdata.patch b/target/linux/ar71xx/patches-4.4/902-at803x-add-reset-gpio-pdata.patch
index 2244f882e7..efc8502bcd 100644
--- a/target/linux/ar71xx/patches-4.4/902-at803x-add-reset-gpio-pdata.patch
+++ b/target/linux/ar71xx/patches-4.4/902-at803x-add-reset-gpio-pdata.patch
@@ -42,7 +42,7 @@ Signed-off-by: Felix Fietkau <nbd at nbd.name>
*/
if (phydev->state == PHY_NOLINK) {
- if (priv->gpiod_reset && !priv->phy_reset) {
-+ if ((priv->gpiod_reset || pdata->has_reset_gpio) &&
++ if ((priv->gpiod_reset || (pdata && pdata->has_reset_gpio)) &&
+ !priv->phy_reset) {
struct at803x_context context;
@@ -52,7 +52,7 @@ Signed-off-by: Felix Fietkau <nbd at nbd.name>
- msleep(1);
- gpiod_set_value(priv->gpiod_reset, 0);
- msleep(1);
-+ if (pdata->has_reset_gpio) {
++ if (pdata && pdata->has_reset_gpio) {
+ gpio_set_value_cansleep(pdata->reset_gpio, 0);
+ msleep(1);
+ gpio_set_value_cansleep(pdata->reset_gpio, 1);
More information about the Lede-dev
mailing list