summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndy Polyakov <appro@openssl.org>2014-03-07 11:02:25 +0100
committerAndy Polyakov <appro@openssl.org>2014-03-07 11:03:51 +0100
commita93d952cd40b7c7276daf9ba604c171275f52a20 (patch)
tree05e08a5efae702325e15ce9d20d57f290e0bbbca
parent5b56fec6f504d21d2de34f9ecdcfe78781e7bfe0 (diff)
downloadopenssl-new-a93d952cd40b7c7276daf9ba604c171275f52a20.tar.gz
engines/ccgost/gosthash.c: simplify and avoid SEGV.
PR: 3275 (cherry picked from commit ea38f020492042bc1d1adc26ef197b0b9cadf968)
-rw-r--r--engines/ccgost/gosthash.c19
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;
}