Applies to 1.0.1 and HEAD. Issue detected by cppcheck.

Signed-off-by: Thomas Jarosch <[email protected]>
---
 engines/ccgost/gost_crypt.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/engines/ccgost/gost_crypt.c b/engines/ccgost/gost_crypt.c
index 4977d1d..cde58c0 100644
--- a/engines/ccgost/gost_crypt.c
+++ b/engines/ccgost/gost_crypt.c
@@ -495,7 +495,8 @@ int  gost89_get_asn1_parameters(EVP_CIPHER_CTX 
*ctx,ASN1_TYPE *params)
 int gost_imit_init_cpa(EVP_MD_CTX *ctx)
        {
        struct ossl_gost_imit_ctx *c = ctx->md_data;
-       memset(c->buffer,0,16);
+       memset(c->buffer,0,sizeof(c->buffer));
+       memset(c->partial_block,0,sizeof(c->partial_block));
        c->count = 0;
        c->bytes_left=0;
        c->key_meshing=1;
-- 
1.7.4.4

>From 23aa91bd3df6f783bf048bdd4a0933d8a3ce019e Mon Sep 17 00:00:00 2001
From: Thomas Jarosch <[email protected]>
Date: Sun, 28 Aug 2011 01:51:12 +0200
Subject: [PATCH 2/4] Don't play games with the struct layout

Applies to 1.0.1 and HEAD. Issue detected by cppcheck.

Signed-off-by: Thomas Jarosch <[email protected]>
---
 engines/ccgost/gost_crypt.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/engines/ccgost/gost_crypt.c b/engines/ccgost/gost_crypt.c
index 4977d1d..cde58c0 100644
--- a/engines/ccgost/gost_crypt.c
+++ b/engines/ccgost/gost_crypt.c
@@ -495,7 +495,8 @@ int  gost89_get_asn1_parameters(EVP_CIPHER_CTX *ctx,ASN1_TYPE *params)
 int gost_imit_init_cpa(EVP_MD_CTX *ctx)
 	{
 	struct ossl_gost_imit_ctx *c = ctx->md_data;
-	memset(c->buffer,0,16);
+	memset(c->buffer,0,sizeof(c->buffer));
+	memset(c->partial_block,0,sizeof(c->partial_block));
 	c->count = 0;
 	c->bytes_left=0;
 	c->key_meshing=1;
-- 
1.7.4.4

Reply via email to