From: Denis Kenzior <denkenz@gmail.com>
To: ell@lists.01.org
Subject: Re: [PATCH 06/13] pkcs5: Add the PKCS#12 KDF
Date: Thu, 10 Dec 2020 22:51:13 +0000 [thread overview]
Message-ID: <183e3ea3-5ed1-4b15-3030-89a53d3eb4d5@gmail.com> (raw)
In-Reply-To: <20201210193202.526451-6-andrew.zaborowski@intel.com>
[-- Attachment #1: Type: text/plain, Size: 1143 bytes --]
Hi Andrew,
On 12/10/20 1:31 PM, Andrew Zaborowski wrote:
> Add the key derivation algorithm used with PKCS#12 to pkcs5.c so that it
> can be found together with the two PKCS#5 KDFs, and so that it can also
> be used when parsing of the PKCS#12 AlgorithmIdentifiers in the next
> commit. This KDF is not recommended for new uses.
> ---
> ell/pkcs5-private.h | 12 ++++
> ell/pkcs5.c | 146 ++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 158 insertions(+)
>
<snip>
> +static bool utf8_to_bmpstring(const char *utf, uint8_t *out_buf,
> + unsigned int *out_len)
> +{
> + unsigned int n;
> + int utflen;
> + wchar_t cp;
> + uint16_t *ptr = (uint16_t *) out_buf;
> +
> + for (n = strlen(utf); n; n -= utflen, utf += utflen) {
> + if ((utflen = l_utf8_get_codepoint(utf, n, &cp)) <= 0 ||
> + cp > 0xffff)
> + return false;
So is this just a UCS2BE conversion? If so, then using l_utf8_to_ucs2be might
be clearer.
> +
> + *ptr++ = L_CPU_TO_BE16(cp);
> + }
> +
> + *ptr++ = 0;
> + *out_len = (uint8_t *) ptr - out_buf;
> + return true;
> +}
> +
Regards,
-Denis
next prev parent reply other threads:[~2020-12-10 22:51 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-12-10 19:31 [PATCH 01/13] pem: Wipe more memory in PKCS#1 private key decoding Andrew Zaborowski
2020-12-10 19:31 ` [PATCH 02/13] pem: Factor out the PKCS#8 decoding code Andrew Zaborowski
2020-12-10 19:31 ` [PATCH 03/13] cipher: Add an RC2 implementation Andrew Zaborowski
2020-12-10 19:31 ` [PATCH 04/13] cipher: Add ARC4 implementation Andrew Zaborowski
2020-12-10 22:46 ` Denis Kenzior
2020-12-10 19:31 ` [PATCH 05/13] pem: Don't expect padding with stream ciphers Andrew Zaborowski
2020-12-10 19:31 ` [PATCH 06/13] pkcs5: Add the PKCS#12 KDF Andrew Zaborowski
2020-12-10 22:51 ` Denis Kenzior [this message]
2020-12-11 10:34 ` Andrew Zaborowski
2020-12-11 15:50 ` Denis Kenzior
2020-12-12 0:06 ` Andrew Zaborowski
2020-12-10 19:32 ` [PATCH 07/13] pkcs5: Add PKCS#12 algorithms in pkcs5_cipher_from_alg_id Andrew Zaborowski
2020-12-10 19:32 ` [PATCH 08/13] pem: Add l_pem_load_container_file Andrew Zaborowski
2020-12-10 19:32 ` [PATCH 09/13] pem: Add PKCS#12 parsing Andrew Zaborowski
2020-12-10 22:47 ` [PATCH 01/13] pem: Wipe more memory in PKCS#1 private key decoding Denis Kenzior
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=183e3ea3-5ed1-4b15-3030-89a53d3eb4d5@gmail.com \
--to=denkenz@gmail.com \
--cc=ell@lists.01.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.