[U-Boot] [PATCH] fix cramfs resolve in case of two files, which start with same chars
Wolfgang Denk
wd at denx.de
Thu Jul 28 11:24:00 CEST 2011
Dear Peter Feuerer,
In message <4E311948.1000203 at sysgo.com> you wrote:
> Hi,
>
> when having two (or more) files in cramfs which start with same name,
> cramfs_resolve didn't check the filename length and thus returned
> potentially the wrong file.
>
> kind regards,
> --peter;
Please move greetings out of the commit message.
> Signed-off-by: Peter Feuerer <pfe at sysgo.com>
>
> ---
> diff -ur u-boot-2011.06_original/fs/cramfs/cramfs.c
> u-boot-2011.06/fs/cramfs/cramfs.c
> --- u-boot-2011.06_original/fs/cramfs/cramfs.c 2011-07-28
> 09:48:10.000000000 +0200
> +++ u-boot-2011.06/fs/cramfs/cramfs.c 2011-07-28 09:51:54.000000000 +0200
> @@ -126,7 +126,8 @@
> namelen--;
> }
>
> - if (!strncmp (filename, name, namelen)) {
> + if (namelen == strlen (filename) &&
> + !strncmp (filename, name, namelen)) {
Please fix all the errors and warnings reported by checkpatch, and
resubmit. Consider using "git format-patch" and "git send-email".
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
It is surely a great calamity for a human being to have no ob-
sessions. - Robert Bly
More information about the U-Boot
mailing list