[U-Boot] [PATCH v3 1/4] mkexynosspl: fix uninitialized buffer

Minkyu Kang mk7.kang at samsung.com
Sat Dec 14 06:07:57 CET 2013


Dear Albert,

On 13/12/13 18:00, Albert ARIBAUD wrote:
> (adding Minkyu as the Samsung custodian)
> 
> On Mon,  9 Dec 2013 18:09:18 +0100, Albert ARIBAUD
> <albert.u.boot at aribaud.net> wrote:
> 
>> mkexynos reads its input file which might be smaller than
> 
> ( typo: s/mkexynos/mkexynosspl/ -- wil fix when applying if thee is
> no other change to be made )
> 
>> its read buffer, but always writes the whole buffer out.
>> This does not affect the functionalyty of the output file,
>> but it makes its content unpredictable as the end of the
>> buffer is never initialized. Fix this by zeroing the buffer
>> before reading the input file.
> 
> Minkyu: should the buffer be pre-filled with zeroes or should I use
> ones instead? Or maybe the output could be truncated to the size of the
> input?

Basically, the output file is fixed to 14K as same as buffer size.
And the input file should be smaller than 14K.
It's the rule of exynos spl.
The role of mkexynosspl is just adding 4 byte checksum value to end of file.
If the input file is bigger than 14K, output will be truncated.

> 
> Amicalement,
> Albert.
>  
>> Signed-off-by: Albert ARIBAUD <albert.u.boot at aribaud.net>

btw, this change looks reasonable to me.

Acked-by: Minkyu Kang <mk7.kang at samsung.com>

>> ---
>> Changes in v3:
>> - fixed mkexynosspl uninitialized buffer
>>
>> Changes in v2: None
>>
>>  tools/mkexynosspl.c | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/tools/mkexynosspl.c b/tools/mkexynosspl.c
>> index ef685b7..8d605fa 100644
>> --- a/tools/mkexynosspl.c
>> +++ b/tools/mkexynosspl.c
>> @@ -72,6 +72,8 @@ int main(int argc, char **argv)
>>  
>>  	count = (len < CHECKSUM_OFFSET) ? len : CHECKSUM_OFFSET;
>>  
>> +	memset(buffer, 0, BUFSIZE);
>> +
>>  	if (read(ifd, buffer, count) != count) {
>>  		fprintf(stderr, "%s: Can't read %s: %s\n",
>>  			argv[0], argv[1], strerror(errno));
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
> 

Thanks,
Minkyu Kang.


More information about the U-Boot mailing list