From: Denis Kenzior <denkenz@gmail.com>
To: Mat Martineau <mathew.j.martineau@linux.intel.com>
Cc: ell@lists.linux.dev
Subject: Re: [PATCH v2 02/10] unit: Add basic EC-DSA verification test
Date: Mon, 18 Jul 2022 15:21:50 -0500 [thread overview]
Message-ID: <83e13273-5c51-815c-99de-6517a42e0b5b@gmail.com> (raw)
In-Reply-To: <559e999-ea7c-cab2-1327-d37b2c36e713@linux.intel.com>
Hi Mat,
> A few typos here: "ce-cert" instead of "ec-cert". This showed up as:
>
Ok, I fixed this in my local copy now.
>
> Other than that, v2 looks good on my system. I didn't try out the kernel patch
Excellent, thanks for testing.
> you mention in patch 1, I assume that's why I had the test failures with
> "tls-suites: Add ECDSA suites from RFC 8422".
The reason is that kernel upstream only supports signature verification and not
the sign operation itself. Not sure why. I'll try to fix that eventually, but
for now we're limited to ECDSA use in anonymous client mode only. The test
assumes that both client + server side is implemented. Hence the need for patch
8/10.
Regards,
-Denis
next prev parent reply other threads:[~2022-07-18 20:24 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-18 18:00 [PATCH v2 01/10] cert/key: Add support for EC based certificates Denis Kenzior
2022-07-18 18:00 ` [PATCH v2 02/10] unit: Add basic EC-DSA verification test Denis Kenzior
2022-07-18 19:07 ` Mat Martineau
2022-07-18 20:21 ` Denis Kenzior [this message]
2022-07-18 18:00 ` [PATCH v2 03/10] key: ECDSA data is given in x962 format Denis Kenzior
2022-07-18 18:00 ` [PATCH v2 04/10] tls: Support peer certificates that use ECDSA Denis Kenzior
2022-07-18 18:00 ` [PATCH v2 05/10] tls: Add helper for DigitallySigned validation Denis Kenzior
2022-07-18 18:00 ` [PATCH v2 06/10] tls: Add helper to find hash function by id Denis Kenzior
2022-07-18 18:00 ` [PATCH v2 07/10] tls-suites: Add ECDSA suites from RFC 8422 Denis Kenzior
2022-07-18 18:00 ` [PATCH v2 08/10] unit: Skip ECDSA cipher suite tests Denis Kenzior
2022-07-18 18:00 ` [PATCH v2 09/10] useful: Add maxsize() Denis Kenzior
2022-07-18 18:00 ` [PATCH v2 10/10] tls: Do not set verify_data_length unless needed 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=83e13273-5c51-815c-99de-6517a42e0b5b@gmail.com \
--to=denkenz@gmail.com \
--cc=ell@lists.linux.dev \
--cc=mathew.j.martineau@linux.intel.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).