On Fri, Jul 27, 2012 at 11:36 AM, Mike Frysinger <vap...@gentoo.org> wrote: > > 2012-07-27 Mike Frysinger <vap...@gentoo.org> > > * md5.c (md5_finish_ctx): Declare swap_bytes. Assign SWAP() output > to swap_bytes, and then call memcpy to move it to ctx->buffer. > > /* Take yet unprocessed bytes into account. */ > - md5_uint32 bytes = ctx->buflen; > + md5_uint32 swap_bytes, bytes = ctx->buflen;
Please add a new line, rather than declaring one uninitialized and one initialized variable on the same line. This patch is OK with that change. Thanks. Ian
