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:02:25 +0100 |
commit | ea38f020492042bc1d1adc26ef197b0b9cadf968 (patch) | |
tree | b46a05285c0aa6d25e0a9eb4c1c6c99fad582b7f /engines | |
parent | 5e44c144e649a53bae2724b34d908f6cb26b01ed (diff) |
engines/ccgost/gosthash.c: simplify and avoid SEGV.
PR: 3275
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 2a458b92ba..a2cba77123 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; } |