[U-Boot] [PATCH] spl: atf: Fix uninitialized pointer to bl31_image_info【请注意，邮件由u-boot-bounces at lists.denx.de代发】 bl31_image_info

[Kever Yang]

On 07/02/2019 10:16 AM, Kever Yang wrote:
> Hi Frieder,
>
>
> On 06/27/2019 03:03 PM, Schrempf Frieder wrote:
>> From: Frieder Schrempf <frieder.schrempf at kontron.de>
>>
>> The pointer to struct atf_image_info in
>> bl31_params_mem.bl31_params.bl31_image_info is not initialized before
>> being dereferenced. This can cause U-Boot to crash right before jumping
>> to the BL31 ATF binary.
>>
>> Signed-off-by: Frieder Schrempf <frieder.schrempf at kontron.de>
>> Fixes: bcc1726a7bdd ("spl: add support to booting with ATF")
> Reviewed-by: Kever Yang <kever.yang at rock-chips.com>

Applied to u-boot-rockchip/master

Thanks,
- Kever
>
> Thanks,
> - Kever
>
>> ---
>>  common/spl/spl_atf.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/common/spl/spl_atf.c b/common/spl/spl_atf.c
>> index cddab6a735..4715f9d371 100644
>> --- a/common/spl/spl_atf.c
>> +++ b/common/spl/spl_atf.c
>> @@ -45,6 +45,7 @@ static struct bl31_params *bl2_plat_get_bl31_params(uintptr_t bl33_entry)
>>  	SET_PARAM_HEAD(bl2_to_bl31_params, ATF_PARAM_BL31, ATF_VERSION_1, 0);
>>  
>>  	/* Fill BL31 related information */
>> +	bl2_to_bl31_params->bl31_image_info = &bl31_params_mem.bl31_image_info;
>>  	SET_PARAM_HEAD(bl2_to_bl31_params->bl31_image_info,
>>  		       ATF_PARAM_IMAGE_BINARY, ATF_VERSION_1, 0);
>>  
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot