[U-Boot] [PATCH 2/3] md5: Fix gcc-4.7 build problem in md5
Wolfgang Denk
wd at denx.de
Sun Nov 4 01:32:42 CET 2012
Dear Simon Glass,
In message <1351979121-3769-2-git-send-email-sjg at chromium.org> you wrote:
> From: Han Shen <shenhan at google.com>
>
> Fixed by replacing pointer casting with memcpy.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> lib/md5.c | 3 +--
> 1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/lib/md5.c b/lib/md5.c
> index 2ae4a06..9791e59 100644
> --- a/lib/md5.c
> +++ b/lib/md5.c
> @@ -153,8 +153,7 @@ MD5Final(unsigned char digest[16], struct MD5Context *ctx)
> byteReverse(ctx->in, 14);
>
> /* Append length in bits and transform */
> - ctx->in32[14] = ctx->bits[0];
> - ctx->in32[15] = ctx->bits[1];
> + memcpy(ctx->in + 14 * sizeof(__u32), ctx->bits, 2 * sizeof(__u32));
This makes the code actually unreadable. Please add at least a
comment what this is doing.
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
Of course there's no reason for it, it's just our policy.
More information about the U-Boot
mailing list