]> git.hungrycats.org Git - linux/commit
staging: vchiq_arm: Get the rid off struct vchiq_2835_state
authorStefan Wahren <wahrenst@gmx.net>
Fri, 21 Jun 2024 13:19:53 +0000 (15:19 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 22 Nov 2024 14:37:32 +0000 (15:37 +0100)
commitc26cff492697cde3c6687a4b62f078170b51e59a
tree1a5cc8f161f2b83edecedf735f9151e61a976294
parentd61a1ccd79b97fb827e5823bfe13e4a248bb8ff3
staging: vchiq_arm: Get the rid off struct vchiq_2835_state

[ Upstream commit 4e2766102da632f26341d5539519b0abf73df887 ]

The whole benefit of this encapsulating struct is questionable.
It just stores a flag to signalize the init state of vchiq_arm_state.
Beside the fact this flag is set too soon, the access to uninitialized
members should be avoided. So initialize vchiq_arm_state properly before
assign it directly to vchiq_state.

Signed-off-by: Stefan Wahren <wahrenst@gmx.net>
Link: https://lore.kernel.org/r/20240621131958.98208-6-wahrenst@gmx.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Stable-dep-of: 404b739e8955 ("staging: vchiq_arm: Use devm_kzalloc() for vchiq_arm_state allocation")
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c