Re: [U-Boot] [PATCH] common/cmd_nvedit.c: Fix size calculation in do_env_import()
Dear KRONSTORFER Horst, In message 024fe3275ef7814d9b07390f28c0832c63a20...@vie196nt.frequentis.frq you wrote: Can you please describe a test case that shows this bug? host: echo foo=bar env.txt mkenvimage -b -s 13 -o env.bin env.txt be target: tftp 20 env.bin env import -c $fileaddr ## Info: input data size = 12 = 0xC ## Error: bad CRC, import failed Thanks. [Note: it would have been even more helpful if you had also provided a hex dump of env.bin - as is, it's difficult for me to see if there is a bug with cmd_nvedit or with your version of the mkenvimage tool.] 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: w...@denx.de Q: Why do PCs have a reset button on the front? A: Because they are expected to run Microsoft operating systems. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH] common/cmd_nvedit.c: Fix size calculation in do_env_import()
do_env_import() missed the final '\0' terminator when calculating the size of an environment data block. This led to an erroneous 'bad CRC, import failed' message for a checksum protected environment (-c.) Signed-off-by: Horst Kronstorfer hkron...@frequentis.com --- common/cmd_nvedit.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/common/cmd_nvedit.c b/common/cmd_nvedit.c index baaa513..63afc82 100644 --- a/common/cmd_nvedit.c +++ b/common/cmd_nvedit.c @@ -868,7 +868,7 @@ static int do_env_import(cmd_tbl_t *cmdtp, int flag, printf(## Warning: Input data exceeds %d bytes - truncated\n, MAX_ENV_SIZE); } - ++size; + size += 2; printf(## Info: input data size = %zu = 0x%zX\n, size, size); } -- 1.7.7.4 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] common/cmd_nvedit.c: Fix size calculation in do_env_import()
Dear Horst Kronstorfer, In message 1324114390-27290-1-git-send-email-hkron...@frequentis.com you wrote: do_env_import() missed the final '\0' terminator when calculating the size of an environment data block. This led to an erroneous 'bad CRC, import failed' message for a checksum protected environment (-c.) Can you please describe a test case that shows this bug? Thanks. 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: w...@denx.de God made the integers; all else is the work of Man. - Kronecker ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot