Add wrappers for reinterpret_cast between char* and uint8_t*

Generally speaking reinterpret_cast is sketchy stuff. But the
special case of char*/uint8_t* is both common and safe. By
isolating those, the remaining (likely sketchy) cases are easier
to grep for.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/lib/kdf/hkdf/hkdf.cpp b/src/lib/kdf/hkdf/hkdf.cpp
index 63085d191..6ccd786c3 100644
--- a/src/lib/kdf/hkdf/hkdf.cpp
+++ b/src/lib/kdf/hkdf/hkdf.cpp
@@ -103,7 +103,7 @@ hkdf_expand_label(const std::string& hash_fn,
    prefix[2] = static_cast<uint8_t>(label.size());
 
    copy_mem(prefix.data() + 3,
-            reinterpret_cast<const uint8_t*>(label.data()),
+            cast_char_ptr_to_uint8(label.data()),
             label.size());
 
    prefix[3 + label.size()] = static_cast<uint8_t>(hash_val_len);