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

Commit 89b620d1 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"... 

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

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



Change-Id: I0a82e481eb0655f6bd56a79dbb567f6f800411f4
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 930c93ed a80338f8

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;

}