commit a80338f80262a94b544874e4b1465595f9f543f9
author Treehugger Robot <treehugger-gerrit@google.com> Fri Aug 19 22:27:19 2022 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Fri Aug 19 22:27:19 2022 +0000
tree 928178431d3a1c03285fa6989e64421db52a30ec
parent e3b2140f8410e4bca25434bedc18000b8fc27e6d
parent a3fe17250dbe2f4222741c9665bf78b487d1d1f3

Merge "Release the memory allocated in the algorithm to prevent memory leaks" am: a3fe17250d

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2186106

Change-Id: Ia83b90d5404dce1dbf8febfa0d66309bce547345
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

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;
 }