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

Commit e8290452 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 am: 89b620d1 am: d6835f63 am: 6def74e9

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



Change-Id: I41e42a4f005bdfb3ddaa2b6cee436727c8195a96
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents e83f5ebc 6def74e9

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;

}