[PATCH-next] i3c: master: Fix potentially uninit variable
Advait Dhamorikar
advaitdhamorikar at gmail.com
Tue Nov 19 07:39:41 PST 2024
devinfo is uninitialized if the condition is not satisfied,
add an else condition to prevent unexpected behaviour.
The variable will contain an arbitrary value left from earlier
computations in `i3c_device_uevent`.
Signed-off-by: Advait Dhamorikar <advaitdhamorikar at gmail.com>
---
drivers/i3c/master.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/i3c/master.c b/drivers/i3c/master.c
index 42310c9a00c2..7594d3793eb0 100644
--- a/drivers/i3c/master.c
+++ b/drivers/i3c/master.c
@@ -284,6 +284,8 @@ static int i3c_device_uevent(const struct device *dev, struct kobj_uevent_env *e
if (i3cdev->desc)
devinfo = i3cdev->desc->info;
+ else
+ return -ENODEV;
manuf = I3C_PID_MANUF_ID(devinfo.pid);
part = I3C_PID_PART_ID(devinfo.pid);
ext = I3C_PID_EXTRA_INFO(devinfo.pid);
--
2.34.1
More information about the linux-i3c
mailing list