From mboxrd@z Thu Jan 1 00:00:00 1970 From: Holger Brunck Date: Mon, 08 Jul 2013 08:53:54 +0200 Subject: [U-Boot] [PATCH] cramfs: fix bug for wrong filename comparison In-Reply-To: <20130705232321.18999465@lilith> References: <1372926586-14495-1-git-send-email-holger.brunck@keymile.com> <20130705232321.18999465@lilith> Message-ID: <51DA6202.6050400@keymile.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 07/05/2013 11:23 PM, Albert ARIBAUD wrote: > On Thu, 4 Jul 2013 10:29:46 +0200, Holger Brunck > 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 >> cc: Wolfgang Denk >> --- > > 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