From bbd1774489f125d99af2345339fdf251815e0b95 Mon Sep 17 00:00:00 2001 From: Shawn Guo Date: Thu, 15 Aug 2019 14:40:24 +0200 Subject: [PATCH] zero-sugar: add a NULL pointer check for max77818_enable_safeout1() Add a NULL pointer check for max77818_enable_safeout1() to make sure we do not call dm_i2c_read[write] with a NULL idDev. Signed-off-by: Shawn Guo --- board/reMarkable/zero-sugar/charger_init.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/board/reMarkable/zero-sugar/charger_init.c b/board/reMarkable/zero-sugar/charger_init.c index 3d56e92e81..f68f8aeb48 100644 --- a/board/reMarkable/zero-sugar/charger_init.c +++ b/board/reMarkable/zero-sugar/charger_init.c @@ -381,6 +381,9 @@ int max77818_enable_safeout1(void) int ret; u8 val; + if (!idDev) + return -ENODEV; + ret = dm_i2c_read(idDev, MAX77818_REG_SAFEOUTCTRL, &val, 1); if (ret) return ret;