Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a80338f8 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

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

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



Change-Id: Ia83b90d5404dce1dbf8febfa0d66309bce547345
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents e3b2140f a3fe1725

identity/aidl/default/EicOpsImpl.cc

+4 −0
Original line number Diff line number Diff line
@@ -100,6 +100,7 @@ void eicOpsHmacSha256Final(EicHmacSha256Ctx* ctx, uint8_t digest[EIC_SHA256_DIGE 
    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 @@ bool eicOpsEcDsa(const uint8_t privateKey[EIC_P256_PRIV_KEY_SIZE], 
    }



    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;

}