commit a3fe17250dbe2f4222741c9665bf78b487d1d1f3
author Treehugger Robot <treehugger-gerrit@google.com> Fri Aug 19 22:09:37 2022 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Fri Aug 19 22:09:37 2022 +0000
tree 928178431d3a1c03285fa6989e64421db52a30ec
parent f533ccb14ebb314f046cca2ebacc4775909b80cd
parent 1aa1cf155ec13bd7e8a40b2f831cfce035b4391d

Merge "Release the memory allocated in the algorithm to prevent memory leaks"

identity/aidl/default/EicOpsImpl.cc

diff --git a/identity/aidl/default/EicOpsImpl.cc b/identity/aidl/default/EicOpsImpl.cc
index 3fd9f1d..b6d324f 100644
--- a/identity/aidl/default/EicOpsImpl.cc
+++ b/identity/aidl/default/EicOpsImpl.cc
@@ -100,6 +100,7 @@
     if (size != EIC_SHA256_DIGEST_SIZE) {
         LOG(ERROR) << "Expected 32 bytes from HMAC_Final, got " << size;
     }
+    HMAC_CTX_cleanup(realCtx);
 }
 
 void eicOpsSha256Init(EicSha256Ctx* ctx) {
@@ -394,14 +395,17 @@
     }
 
     if (BN_bn2binpad(sig->r, signature, 32) != 32) {
+        ECDSA_SIG_free(sig);
         eicDebug("Error encoding r");
         return false;
     }
     if (BN_bn2binpad(sig->s, signature + 32, 32) != 32) {
+        ECDSA_SIG_free(sig);
         eicDebug("Error encoding s");
         return false;
     }
 
+    ECDSA_SIG_free(sig);
     return true;
 }