rsaenh: Remove dead assignments (Clang)
Nikolay Sivov
bunglehead at gmail.com
Sun Nov 29 16:39:09 CST 2015
On 30.11.2015 0:31, André Hentschel wrote:
> Signed-off-by: André Hentschel <nerv at dawncrow.de>
> ---
> dlls/rsaenh/sha2.c | 21 ---------------------
> 1 file changed, 21 deletions(-)
>
> diff --git a/dlls/rsaenh/sha2.c b/dlls/rsaenh/sha2.c
> index 76e7184..71ebf23 100644
> --- a/dlls/rsaenh/sha2.c
> +++ b/dlls/rsaenh/sha2.c
> @@ -369,9 +369,6 @@ void SHA256_Transform(SHA256_CTX* context, const sha2_word32* data) {
> context->state[5] += f;
> context->state[6] += g;
> context->state[7] += h;
> -
> - /* Clean up */
> - a = b = c = d = e = f = g = h = T1 = 0;
> }
>
> #else /* SHA2_UNROLL_TRANSFORM */
> @@ -449,9 +446,6 @@ void SHA256_Transform(SHA256_CTX* context, const sha2_word32* data) {
> context->state[5] += f;
> context->state[6] += g;
> context->state[7] += h;
> -
> - /* Clean up */
> - a = b = c = d = e = f = g = h = T1 = T2 = 0;
> }
>
> #endif /* SHA2_UNROLL_TRANSFORM */
> @@ -483,8 +477,6 @@ void SHA256_Update(SHA256_CTX* context, const sha2_byte *data, size_t len) {
> /* The buffer is not yet full */
> MEMCPY_BCOPY(&context->buffer[usedspace], data, len);
> context->bitcount += len << 3;
> - /* Clean up: */
> - usedspace = freespace = 0;
> return;
> }
> }
> @@ -500,8 +492,6 @@ void SHA256_Update(SHA256_CTX* context, const sha2_byte *data, size_t len) {
> MEMCPY_BCOPY(context->buffer, data, len);
> context->bitcount += len << 3;
> }
> - /* Clean up: */
> - usedspace = freespace = 0;
> }
>
> void SHA256_Final(sha2_byte digest[], SHA256_CTX* context) {
> @@ -564,7 +554,6 @@ void SHA256_Final(sha2_byte digest[], SHA256_CTX* context) {
>
> /* Clean up state data: */
> MEMSET_BZERO(context, sizeof(*context));
> - usedspace = 0;
> }
>
> char *SHA256_End(SHA256_CTX* context, char buffer[]) {
> @@ -693,9 +682,6 @@ void SHA512_Transform(SHA512_CTX* context, const sha2_word64* data) {
> context->state[5] += f;
> context->state[6] += g;
> context->state[7] += h;
> -
> - /* Clean up */
> - a = b = c = d = e = f = g = h = T1 = 0;
> }
>
> #else /* SHA2_UNROLL_TRANSFORM */
> @@ -771,9 +757,6 @@ void SHA512_Transform(SHA512_CTX* context, const sha2_word64* data) {
> context->state[5] += f;
> context->state[6] += g;
> context->state[7] += h;
> -
> - /* Clean up */
> - a = b = c = d = e = f = g = h = T1 = T2 = 0;
> }
>
> #endif /* SHA2_UNROLL_TRANSFORM */
> @@ -805,8 +788,6 @@ void SHA512_Update(SHA512_CTX* context, const sha2_byte *data, size_t len) {
> /* The buffer is not yet full */
> MEMCPY_BCOPY(&context->buffer[usedspace], data, len);
> ADDINC128(context->bitcount, len << 3);
> - /* Clean up: */
> - usedspace = freespace = 0;
> return;
> }
> }
> @@ -822,8 +803,6 @@ void SHA512_Update(SHA512_CTX* context, const sha2_byte *data, size_t len) {
> MEMCPY_BCOPY(context->buffer, data, len);
> ADDINC128(context->bitcount, len << 3);
> }
> - /* Clean up: */
> - usedspace = freespace = 0;
> }
>
> void SHA512_Last(SHA512_CTX* context) {
>
This and advapi32 looks like was done on purpose. Why are you removing that?
More information about the wine-devel
mailing list