* [PATCH] crypto: testmgr - fix version number of RSA tests
@ 2022-06-15 9:13 Lei He
2022-06-24 1:53 ` PING: " 何磊
0 siblings, 1 reply; 7+ messages in thread
From: Lei He @ 2022-06-15 9:13 UTC (permalink / raw)
To: herbert, davem, linux-crypto, linux-kernel; +Cc: helei.sig11, pizhenwei
From: lei he <helei.sig11@bytedance.com>
According to PKCS#1 standard, the 'otherPrimeInfos' field contains
the information for the additional primes r_3, ..., r_u, in order.
It shall be omitted if the version is 0 and shall contain at least
one instance of OtherPrimeInfo if the version is 1, see:
https://www.rfc-editor.org/rfc/rfc3447#page-44
Replace the version number '1' with 0, otherwise, some drivers may
not pass the run-time tests.
Signed-off-by: lei he <helei.sig11@bytedance.com>
---
crypto/testmgr.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/crypto/testmgr.h b/crypto/testmgr.h
index 4d7449fc6a65..d57f24b906f1 100644
--- a/crypto/testmgr.h
+++ b/crypto/testmgr.h
@@ -186,7 +186,7 @@ static const struct akcipher_testvec rsa_tv_template[] = {
#ifndef CONFIG_CRYPTO_FIPS
.key =
"\x30\x81\x9A" /* sequence of 154 bytes */
- "\x02\x01\x01" /* version - integer of 1 byte */
+ "\x02\x01\x00" /* version - integer of 1 byte */
"\x02\x41" /* modulus - integer of 65 bytes */
"\x00\xAA\x36\xAB\xCE\x88\xAC\xFD\xFF\x55\x52\x3C\x7F\xC4\x52\x3F"
"\x90\xEF\xA0\x0D\xF3\x77\x4A\x25\x9F\x2E\x62\xB4\xC5\xD9\x9C\xB5"
@@ -216,7 +216,7 @@ static const struct akcipher_testvec rsa_tv_template[] = {
}, {
.key =
"\x30\x82\x01\x1D" /* sequence of 285 bytes */
- "\x02\x01\x01" /* version - integer of 1 byte */
+ "\x02\x01\x00" /* version - integer of 1 byte */
"\x02\x81\x81" /* modulus - integer of 129 bytes */
"\x00\xBB\xF8\x2F\x09\x06\x82\xCE\x9C\x23\x38\xAC\x2B\x9D\xA8\x71"
"\xF7\x36\x8D\x07\xEE\xD4\x10\x43\xA4\x40\xD6\xB6\xF0\x74\x54\xF5"
@@ -260,7 +260,7 @@ static const struct akcipher_testvec rsa_tv_template[] = {
#endif
.key =
"\x30\x82\x02\x20" /* sequence of 544 bytes */
- "\x02\x01\x01" /* version - integer of 1 byte */
+ "\x02\x01\x00" /* version - integer of 1 byte */
"\x02\x82\x01\x01\x00" /* modulus - integer of 256 bytes */
"\xDB\x10\x1A\xC2\xA3\xF1\xDC\xFF\x13\x6B\xED\x44\xDF\xF0\x02\x6D"
"\x13\xC7\x88\xDA\x70\x6B\x54\xF1\xE8\x27\xDC\xC3\x0F\x99\x6A\xFA"
--
2.20.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* PING: [PATCH] crypto: testmgr - fix version number of RSA tests
2022-06-15 9:13 [PATCH] crypto: testmgr - fix version number of RSA tests Lei He
@ 2022-06-24 1:53 ` 何磊
2022-06-24 8:54 ` Herbert Xu
0 siblings, 1 reply; 7+ messages in thread
From: 何磊 @ 2022-06-24 1:53 UTC (permalink / raw)
To: herbert, davem, linux-crypto, linux-kernel; +Cc: 何磊, pizhenwei
PING！
> On Jun 15, 2022, at 5:13 PM, Lei He <helei.sig11@bytedance.com> wrote:
>
> From: lei he <helei.sig11@bytedance.com>
>
> According to PKCS#1 standard, the 'otherPrimeInfos' field contains
> the information for the additional primes r_3, ..., r_u, in order.
> It shall be omitted if the version is 0 and shall contain at least
> one instance of OtherPrimeInfo if the version is 1, see:
> https://www.rfc-editor.org/rfc/rfc3447#page-44
>
> Replace the version number '1' with 0, otherwise, some drivers may
> not pass the run-time tests.
>
> Signed-off-by: lei he <helei.sig11@bytedance.com>
> ---
> crypto/testmgr.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/crypto/testmgr.h b/crypto/testmgr.h
> index 4d7449fc6a65..d57f24b906f1 100644
> --- a/crypto/testmgr.h
> +++ b/crypto/testmgr.h
> @@ -186,7 +186,7 @@ static const struct akcipher_testvec rsa_tv_template[] = {
> #ifndef CONFIG_CRYPTO_FIPS
> .key =
> "\x30\x81\x9A" /* sequence of 154 bytes */
> - "\x02\x01\x01" /* version - integer of 1 byte */
> + "\x02\x01\x00" /* version - integer of 1 byte */
> "\x02\x41" /* modulus - integer of 65 bytes */
> "\x00\xAA\x36\xAB\xCE\x88\xAC\xFD\xFF\x55\x52\x3C\x7F\xC4\x52\x3F"
> "\x90\xEF\xA0\x0D\xF3\x77\x4A\x25\x9F\x2E\x62\xB4\xC5\xD9\x9C\xB5"
> @@ -216,7 +216,7 @@ static const struct akcipher_testvec rsa_tv_template[] = {
> }, {
> .key =
> "\x30\x82\x01\x1D" /* sequence of 285 bytes */
> - "\x02\x01\x01" /* version - integer of 1 byte */
> + "\x02\x01\x00" /* version - integer of 1 byte */
> "\x02\x81\x81" /* modulus - integer of 129 bytes */
> "\x00\xBB\xF8\x2F\x09\x06\x82\xCE\x9C\x23\x38\xAC\x2B\x9D\xA8\x71"
> "\xF7\x36\x8D\x07\xEE\xD4\x10\x43\xA4\x40\xD6\xB6\xF0\x74\x54\xF5"
> @@ -260,7 +260,7 @@ static const struct akcipher_testvec rsa_tv_template[] = {
> #endif
> .key =
> "\x30\x82\x02\x20" /* sequence of 544 bytes */
> - "\x02\x01\x01" /* version - integer of 1 byte */
> + "\x02\x01\x00" /* version - integer of 1 byte */
> "\x02\x82\x01\x01\x00" /* modulus - integer of 256 bytes */
> "\xDB\x10\x1A\xC2\xA3\xF1\xDC\xFF\x13\x6B\xED\x44\xDF\xF0\x02\x6D"
> "\x13\xC7\x88\xDA\x70\x6B\x54\xF1\xE8\x27\xDC\xC3\x0F\x99\x6A\xFA"
> --
> 2.20.1
>
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: PING: [PATCH] crypto: testmgr - fix version number of RSA tests
2022-06-24 1:53 ` PING: " 何磊
@ 2022-06-24 8:54 ` Herbert Xu
2022-06-24 10:29 ` [External] " 何磊
0 siblings, 1 reply; 7+ messages in thread
From: Herbert Xu @ 2022-06-24 8:54 UTC (permalink / raw)
To: 何磊; +Cc: davem, linux-crypto, linux-kernel, pizhenwei
On Fri, Jun 24, 2022 at 09:53:02AM +0800, 何磊 wrote:
> PING！
Please resubmit.
Thanks,
--
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [External] PING: [PATCH] crypto: testmgr - fix version number of RSA tests
2022-06-24 8:54 ` Herbert Xu
@ 2022-06-24 10:29 ` 何磊
2022-06-25 0:32 ` Herbert Xu
0 siblings, 1 reply; 7+ messages in thread
From: 何磊 @ 2022-06-24 10:29 UTC (permalink / raw)
To: Herbert Xu; +Cc: 何磊, davem, linux-crypto, linux-kernel, pizhenwei
> On Jun 24, 2022, at 4:54 PM, Herbert Xu <herbert@gondor.apana.org.au> wrote:
>
> On Fri, Jun 24, 2022 at 09:53:02AM +0800, 何磊 wrote:
>> PING！
>
> Please resubmit.
>
Thanks a lot for your reply, a new patch has been sent.
By the way, why this patch needs to be resubmitted. Please let me know if I have made any mistakes.
> Thanks,
> --
> Email: Herbert Xu <herbert@gondor.apana.org.au>
> Home Page: http://gondor.apana.org.au/~herbert/
> PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [External] PING: [PATCH] crypto: testmgr - fix version number of RSA tests
2022-06-24 10:29 ` [External] " 何磊
@ 2022-06-25 0:32 ` Herbert Xu
2022-06-27 6:16 ` Lei He
0 siblings, 1 reply; 7+ messages in thread
From: Herbert Xu @ 2022-06-25 0:32 UTC (permalink / raw)
To: 何磊; +Cc: davem, linux-crypto, linux-kernel, pizhenwei
On Fri, Jun 24, 2022 at 06:29:29PM +0800, 何磊 wrote:
>
>
> > On Jun 24, 2022, at 4:54 PM, Herbert Xu <herbert@gondor.apana.org.au> wrote:
> >
> > On Fri, Jun 24, 2022 at 09:53:02AM +0800, 何磊 wrote:
> >> PING！
> >
> > Please resubmit.
> >
>
> Thanks a lot for your reply, a new patch has been sent.
> By the way, why this patch needs to be resubmitted. Please let me know if I have made any mistakes.
You first sent a subsequent version that superceded the
original patch. That subsequent patch was then dismissed because
you replied in the thread saying that it needed changes.
Please be more careful in how you send patches and thread them.
Cheers,
--
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [External] PING: [PATCH] crypto: testmgr - fix version number of RSA tests
2022-06-25 0:32 ` Herbert Xu
@ 2022-06-27 6:16 ` Lei He
2022-06-27 6:47 ` Lei He
0 siblings, 1 reply; 7+ messages in thread
From: Lei He @ 2022-06-27 6:16 UTC (permalink / raw)
To: Herbert Xu; +Cc: Lei He, davem, linux-crypto, linux-kernel, pizhenwei
> On Jun 25, 2022, at 8:32 AM, Herbert Xu <herbert@gondor.apana.org.au> wrote:
>
> On Fri, Jun 24, 2022 at 06:29:29PM +0800, 何磊 wrote:
>>
>>
>>> On Jun 24, 2022, at 4:54 PM, Herbert Xu <herbert@gondor.apana.org.au> wrote:
>>>
>>> On Fri, Jun 24, 2022 at 09:53:02AM +0800, 何磊 wrote:
>>>> PING！
>>>
>>> Please resubmit.
>>>
>>
>> Thanks a lot for your reply, a new patch has been sent.
>> By the way, why this patch needs to be resubmitted. Please let me know if I have made any mistakes.
>
> You first sent a subsequent version that superceded the
> original patch. That subsequent patch was then dismissed because
> you replied in the thread saying that it needed changes.
>
> Please be more careful in how you send patches and thread them.
Thanks for the explanation, now all patches have been resubmitted. Sorry
for the extra trouble, I'll be more careful in the future.
>
> Cheers,
> --
> Email: Herbert Xu <herbert@gondor.apana.org.au>
> Home Page: http://gondor.apana.org.au/~herbert/
> PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [External] PING: [PATCH] crypto: testmgr - fix version number of RSA tests
2022-06-27 6:16 ` Lei He
@ 2022-06-27 6:47 ` Lei He
0 siblings, 0 replies; 7+ messages in thread
From: Lei He @ 2022-06-27 6:47 UTC (permalink / raw)
To: Herbert Xu; +Cc: Lei He, davem, linux-crypto, linux-kernel, pizhenwei
> On Jun 27, 2022, at 2:16 PM, Lei He <helei.sig11@bytedance.com> wrote:
>
>>
>> On Jun 25, 2022, at 8:32 AM, Herbert Xu <herbert@gondor.apana.org.au> wrote:
>>
>> On Fri, Jun 24, 2022 at 06:29:29PM +0800, 何磊 wrote:
>>>
>>>
>>>> On Jun 24, 2022, at 4:54 PM, Herbert Xu <herbert@gondor.apana.org.au> wrote:
>>>>
>>>> On Fri, Jun 24, 2022 at 09:53:02AM +0800, 何磊 wrote:
>>>>> PING！
>>>>
>>>> Please resubmit.
>>>>
>>>
>>> Thanks a lot for your reply, a new patch has been sent.
>>> By the way, why this patch needs to be resubmitted. Please let me know if I have made any mistakes.
>>
>> You first sent a subsequent version that superceded the
>> original patch. That subsequent patch was then dismissed because
>> you replied in the thread saying that it needed changes.
>>
>> Please be more careful in how you send patches and thread them.
>
> Thanks for the explanation, now all patches have been resubmitted. Sorry
> for the extra trouble, I'll be more careful in the future.
Here are the links to the latest patches:
https://lkml.org/lkml/2022/6/23/150
https://lkml.org/lkml/2022/6/24/357
>
>>
>> Cheers,
>> --
>> Email: Herbert Xu <herbert@gondor.apana.org.au>
>> Home Page: http://gondor.apana.org.au/~herbert/
>> PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-06-27 6:47 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-15 9:13 [PATCH] crypto: testmgr - fix version number of RSA tests Lei He
2022-06-24 1:53 ` PING: " 何磊
2022-06-24 8:54 ` Herbert Xu
2022-06-24 10:29 ` [External] " 何磊
2022-06-25 0:32 ` Herbert Xu
2022-06-27 6:16 ` Lei He
2022-06-27 6:47 ` Lei He
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.