On 2/14/2025 9:21 AM, Jeffrey Hugo wrote:
If mhi_fw_load_handler() bails out early because the EE is not capable
of loading firmware, we may reference fw_load_type in cleanup which is
uninitialized at this point. The cleanup code checks fw_load_type as a
proxy for knowing if fbc_image was allocated and needs to be freed, but
we can directly test for that. This avoids the possible uninitialized
access and appears to be clearer code.
Reported-by: Dan Carpenter <[email protected]>
Closes:
https://lore.kernel.org/all/[email protected]/
Fixes: f88f1d0998ea ("bus: mhi: host: Add a policy to enable image transfer via BHIe
in PBL")
Signed-off-by: Jeffrey Hugo <[email protected]>
Acked-by: Manivannan Sadhasivam <[email protected]>
Reviewed-by: Carl Vanderlip <[email protected]>
Applied to drm-misc-next.
-Jeff