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/pbkdf/pgp_s2k/pgp_s2k.cpp b/src/lib/pbkdf/pgp_s2k/pgp_s2k.cpp
index f298bb3e4..14f21eae2 100644
--- a/src/lib/pbkdf/pgp_s2k/pgp_s2k.cpp
+++ b/src/lib/pbkdf/pgp_s2k/pgp_s2k.cpp
@@ -91,7 +91,7 @@ size_t OpenPGP_S2K::pbkdf(uint8_t output_buf[], size_t output_len,
    if(passphrase.empty() == false)
       {
       copy_mem(&input_buf[salt_len],
-               reinterpret_cast<const uint8_t*>(passphrase.data()),
+               cast_char_ptr_to_uint8(passphrase.data()),
                passphrase.size());
       }