Avoid calling memset, memcpy within library code

Prefer using wrappers in mem_utils for this.

Current exception is where memcpy is being used to convert between
two different types, since copy_mem requires input and output
pointers have the same type. There should be a new function to
handle conversion-via-memcpy operation.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/lib/prov/tpm/tpm.cpp b/src/lib/prov/tpm/tpm.cpp
index 4847b0df6..ffbc220d6 100644
--- a/src/lib/prov/tpm/tpm.cpp
+++ b/src/lib/prov/tpm/tpm.cpp
@@ -191,7 +191,7 @@ void TPM_Context::gen_random(uint8_t out[], size_t out_len)
    {
    BYTE* mem;
    TSPI_CHECK_SUCCESS(::Tspi_TPM_GetRandom(m_tpm, out_len, &mem));
-   std::memcpy(out, mem, out_len);
+   copy_mem(out, reinterpret_cast<const uint8_t*>(mem), out_len);
    TSPI_CHECK_SUCCESS(::Tspi_Context_FreeMemory(m_ctx, mem));
    }