diff options
author | Andy Polyakov <appro@openssl.org> | 2014-03-07 11:02:25 +0100 |
---|---|---|
committer | Andy Polyakov <appro@openssl.org> | 2014-03-07 11:04:10 +0100 |
commit | ce2c6d845217f67a51007e43c703dbca0529634c (patch) | |
tree | fdfef83979920acffa7e6efdf2c8b4a07a161925 /engines | |
parent | e67c71c8fd0567d2fb7fda10a678a28ee6d6c9b2 (diff) |
engines/ccgost/gosthash.c: simplify and avoid SEGV.
PR: 3275
(cherry picked from commit ea38f020492042bc1d1adc26ef197b0b9cadf968)
Diffstat (limited to 'engines')
-rw-r--r-- | engines/ccgost/gosthash.c | 19 |
1 files changed, 9 insertions, 10 deletions
diff --git a/engines/ccgost/gosthash.c b/engines/ccgost/gosthash.c index 8c278aa645..91b2ce8829 100644 --- a/engines/ccgost/gosthash.c +++ b/engines/ccgost/gosthash.c @@ -180,8 +180,6 @@ int start_hash(gost_hash_ctx *ctx) */ int hash_block(gost_hash_ctx *ctx,const byte *block, size_t length) { - const byte *curptr=block; - const byte *barrier=block+(length-32);/* Last byte we can safely hash*/ if (ctx->left) { /*There are some bytes from previous step*/ @@ -196,24 +194,25 @@ int hash_block(gost_hash_ctx *ctx,const byte *block, size_t length) { return 1; } - curptr=block+add_bytes; + block+=add_bytes; + length-=add_bytes; hash_step(ctx->cipher_ctx,ctx->H,ctx->remainder); add_blocks(32,ctx->S,ctx->remainder); ctx->len+=32; ctx->left=0; } - while (curptr<=barrier) + while (length>=32) { - hash_step(ctx->cipher_ctx,ctx->H,curptr); + hash_step(ctx->cipher_ctx,ctx->H,block); - add_blocks(32,ctx->S,curptr); + add_blocks(32,ctx->S,block); ctx->len+=32; - curptr+=32; + block+=32; + length-=32; } - if (curptr!=block+length) + if (length) { - ctx->left=block+length-curptr; - memcpy(ctx->remainder,curptr,ctx->left); + memcpy(ctx->remainder,block,ctx->left=length); } return 1; } |