[U-Boot] [PATCH] cramfs: fix bug for wrong filename comparison

[Holger Brunck]

On 07/05/2013 11:23 PM, Albert ARIBAUD wrote:
> On Thu,  4 Jul 2013 10:29:46 +0200, Holger Brunck
> <holger.brunck at keymile.com> wrote:
> 
>> If we have the following entry in cramfs:
>> => cramfsls
>>  -rw-r--r--  1922689 uImage
>>
>> cramfsload would also succeed if we try to do:
>> => cramfsload uImage_1
>> CRAMFS load complete: 1922689 bytes loaded to 0x100000
>>
>> The old code succeeds if the begin of the filename we search matches
>> with a filename stored in cramfs. But the searched file may have
>> additional characters and is therfore not the file we are looking for.
>> So compare also the length of the filename we search and the
>> filename we found in cramfs. This leads to:
>> => cramfsload uImage_1
>> can't find corresponding entry
>> CRAMFS LOAD ERROR<0> for uImage_1!
>>
>> which is the behaviour we want.
>> Signed-off-by: Holger Brunck <holger.brunck at keymile.com>
>> cc: Wolfgang Denk <wd at denx.de>
>> ---
> 
> Can't the commit message above be summarized as follows?
> 
> ---8<---
> "cramfsload uImage_1" succeeds even though the actual file is named
> "uImage".
> 
> Fix file name comparison when one name is the prefix of the other.
> ---8<---
> 
> The demonstrative part of the commit message can go here, below the
> commit message delimiter '---'.
> 

ok. I'll send a v2.

Regards
Holger