From dbbaccc781ad78c8d7a310c25c73383d88b57dcc Mon Sep 17 00:00:00 2001 From: Bruno Meneguele Date: Wed, 15 Jul 2020 18:39:05 -0300 Subject: [PATCH] ima-evm-utils: fix memory leak in case of error OpenSSL context should be freed in case of versions >= 1.1 before leaving the function in case EVP_DigestUpdate() returns any error. Signed-off-by: Bruno Meneguele Signed-off-by: Mimi Zohar --- src/evmctl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/evmctl.c b/src/evmctl.c index f53ba73..e5140c5 100644 --- a/src/evmctl.c +++ b/src/evmctl.c @@ -2156,7 +2156,7 @@ static void calc_bootaggr(struct tpm_bank_info *bank) err = EVP_DigestUpdate(pctx, bank->pcr[i], bank->digest_size); if (!err) { log_err("EVP_DigestUpdate() failed\n"); - return; + goto out; } } @@ -2165,7 +2165,7 @@ static void calc_bootaggr(struct tpm_bank_info *bank) err = EVP_DigestUpdate(pctx, bank->pcr[i], bank->digest_size); if (!err) { log_err("EVP_DigestUpdate() failed\n"); - return; + goto out; } } }