diff options
author | Pauli <paul.dale@oracle.com> | 2020-02-05 15:13:49 +1000 |
---|---|---|
committer | Pauli <paul.dale@oracle.com> | 2020-02-06 16:22:00 +1000 |
commit | 8b6ffd40401bd3b78538cb8d496db0c6926185b0 (patch) | |
tree | 6329353eb7e6c01592b28898974ae2b8f2faaf9b /crypto/dh/dh_kdf.c | |
parent | b744f915ca8bb37631909728dd2529289bda8438 (diff) |
Params: change UTF8 construct calls to avoid explicit strlen(3) calls.
It is better, safer and smaller to let the library routine handle the
strlen(3) call.
Added a note to the documentation suggesting this.
Reviewed-by: Tim Hudson <tjh@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/11019)
Diffstat (limited to 'crypto/dh/dh_kdf.c')
-rw-r--r-- | crypto/dh/dh_kdf.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/crypto/dh/dh_kdf.c b/crypto/dh/dh_kdf.c index a1bbea3013..23bc8acb73 100644 --- a/crypto/dh/dh_kdf.c +++ b/crypto/dh/dh_kdf.c @@ -43,14 +43,14 @@ int DH_KDF_X9_42(unsigned char *out, size_t outlen, if ((kctx = EVP_KDF_CTX_new(kdf)) == NULL) goto err; *p++ = OSSL_PARAM_construct_utf8_string(OSSL_KDF_PARAM_DIGEST, - (char *)mdname, strlen(mdname) + 1); + (char *)mdname, 0); *p++ = OSSL_PARAM_construct_octet_string(OSSL_KDF_PARAM_KEY, (unsigned char *)Z, Zlen); if (ukm != NULL) *p++ = OSSL_PARAM_construct_octet_string(OSSL_KDF_PARAM_UKM, (unsigned char *)ukm, ukmlen); *p++ = OSSL_PARAM_construct_utf8_string(OSSL_KDF_PARAM_CEK_ALG, - (char *)oid_sn, strlen(oid_sn) + 1); + (char *)oid_sn, 0); *p = OSSL_PARAM_construct_end(); ret = EVP_KDF_CTX_set_params(kctx, params) > 0 && EVP_KDF_derive(kctx, out, outlen) > 0; |