[PATCH] drm/bridge: it6505: Fix potential NULL dereference

From: Dan Carpenter
Date: Sat Jun 08 2024 - 10:21:22 EST


Smatch complains correctly that the NULL checking isn't consistent:

drivers/gpu/drm/bridge/ite-it6505.c:2583 it6505_poweron()
error: we previously assumed 'pdata->pwr18' could be null
(see line 2569)

Add a NULL check to prevent a NULL dereference on the error path.

Fixes: b5c84a9edcd4 ("drm/bridge: add it6505 driver")
Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
---
drivers/gpu/drm/bridge/ite-it6505.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/bridge/ite-it6505.c b/drivers/gpu/drm/bridge/ite-it6505.c
index 3f68c82888c2..4f01fadaec0f 100644
--- a/drivers/gpu/drm/bridge/ite-it6505.c
+++ b/drivers/gpu/drm/bridge/ite-it6505.c
@@ -2580,7 +2580,8 @@ static int it6505_poweron(struct it6505 *it6505)
usleep_range(1000, 2000);
err = regulator_enable(pdata->ovdd);
if (err) {
- regulator_disable(pdata->pwr18);
+ if (pdata->pwr18)
+ regulator_disable(pdata->pwr18);
return err;
}
}
--
2.43.0