[U-Boot] [PATCH 1/1] imximage: header.length of 4 is valid

[stefano babic <sbabic@denx.de>]

Hi Troy,

Am 06.10.2015 um 00:17 schrieb Troy Kisky:
> On 9/21/2015 2:02 PM, Troy Kisky wrote:
>> Signed-off-by: Troy Kisky <troy.kisky@boundarydevices.com>
>> ---
>>  tools/imximage.c | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/tools/imximage.c b/tools/imximage.c
>> index 97a6880..7c21922 100644
>> --- a/tools/imximage.c
>> +++ b/tools/imximage.c
>> @@ -396,8 +396,8 @@ static void print_hdr_v2(struct imx_header *imx_hdr)
>>  	dcd_v2_t *dcd_v2 = &hdr_v2->dcd_table;
>>  	uint32_t size, version;
>>  
>> -	size = be16_to_cpu(dcd_v2->header.length) - 8;
>> -	if (size > (MAX_HW_CFG_SIZE_V2 * sizeof(dcd_addr_data_t))) {
>> +	size = be16_to_cpu(dcd_v2->header.length);
>> +	if (size > (MAX_HW_CFG_SIZE_V2 * sizeof(dcd_addr_data_t)) + 8) {
>>  		fprintf(stderr,
>>  			"Error: Image corrupt DCD size %d exceed maximum %d\n",
>>  			(uint32_t)(size / sizeof(dcd_addr_data_t)),
>>
> Hi Stefano,
>
>
> Are there outstanding concerns about applying this patch and then
> applying
>
> imximage: fix commands other than write_data
>
> Where you replied
> "This patch breaks building boards with SPL:
>
> Building current source for 85 boards (6 threads, 1 job per thread)
>        arm:  +   colibri_vf_dtb
> +Error: Image corrupt DCD size 536870911 exceed maximum 220
> +make[2]: *** [u-boot-dtb.imx] Error 1
> +make[1]: *** [u-boot-dtb.imx] Error 2
> +make: *** [sub-make] Error 2
>        arm:  +   mx6sabresd_spl
> +Error: Image corrupt DCD size 536870911 exceed maximum 220
> "
>
>
> Or would you rather that I squash them together?

Yes, please do it - I would like to avoid issues by git bisect.

Thanks,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================