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

Commit d6835f63 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

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



Change-Id: I0d9fadac7dfe9b94340fa900e744258aa6eda345
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 599f7c01 89b620d1

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;

}