From da0c648be057d9d9277a5a17411d4acb5dfe710f Mon Sep 17 00:00:00 2001 From: Vitaly Chikunov Date: Mon, 10 Jan 2022 04:27:00 +0300 Subject: [PATCH] gost_prov: OPENSSL_free what is OPENSSL_zalloc'd *** CID 345254: API usage errors (ALLOC_FREE_MISMATCH) /gost_prov.c: 71 in provider_ctx_new() 65 && populate_gost_engine(ctx->e)) { 66 ctx->core_handle = core; 67 68 /* Ugly hack */ 69 err_handle = ctx->proverr_handle; 70 } else { >>> CID 345254: API usage errors (ALLOC_FREE_MISMATCH) >>> Calling "provider_ctx_free" frees "ctx" using "free" but it should have been freed using "CRYPTO_free". 71 provider_ctx_free(ctx); 72 ctx = NULL; 73 } 74 return ctx; 75 } Fixes: f5a3951 ("gost_prov: Avoid access to unallocated memory") Signed-off-by: Vitaly Chikunov --- gost_prov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gost_prov.c b/gost_prov.c index 3d23765..eab5918 100644 --- a/gost_prov.c +++ b/gost_prov.c @@ -49,7 +49,7 @@ static void provider_ctx_free(PROV_CTX *ctx) proverr_free_handle(ctx->proverr_handle); OSSL_LIB_CTX_free(ctx->libctx); } - free(ctx); + OPENSSL_free(ctx); } extern int populate_gost_engine(ENGINE *e); -- 2.39.2