net/data/verify_signed_data_unittest/rsa-pkcs1-sha256-using-ecdsa-algorithm.pem - Issue 1209283004: Implement VerifySignedData() for ECDSA, RSA PKCS#1 and RSA PSS.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: net/data/verify_signed_data_unittest/rsa-pkcs1-sha256-using-ecdsa-algorithm.pem

Issue 1209283004: Implement VerifySignedData() for ECDSA, RSA PKCS#1 and RSA PSS. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@parse_pss

Patch Set: add a test for non-BIT STRING signature value Created 5 years, 5 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

« net/cert/internal/verify_signed_data.cc ('K') | « net/data/verify_signed_data_unittest/rsa-pkcs1-sha256-spki-non-null-params.pem ('k') | net/data/verify_signed_data_unittest/rsa-pkcs1-sha256-using-id-ea-rsa.pem » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
(Empty)
	1 This test specified a valid RSA PKCS#1 v.1.5 signature and RSA key (the same as rsa-pkcs1-sha256.pem).

	2

	3 The problem however is the signature algorithm is indicated as being ECDSA.

	4

	5 Signature verification consequently should fail.

	6

	7

	8 $ openssl asn1parse -i < [PUBLIC KEY]

	9 0:d=0 hl=3 l= 159 cons: SEQUENCE

	10 3:d=1 hl=2 l= 13 cons: SEQUENCE

	11 5:d=2 hl=2 l= 9 prim: OBJECT :rsaEncryption

	12 16:d=2 hl=2 l= 0 prim: NULL

	13 18:d=1 hl=3 l= 141 prim: BIT STRING

	14 -----BEGIN PUBLIC KEY-----

	15 MIGfMA0GCSqGSIb3DQEBAQUAA4GNADCBiQKBgQCqR+B2Mj1irNp1JnY2zQIQRQPz7ybs6mUjHT3

	16 hf5APyaig2u6fBPThqxssgg0YviqIj/70hVK/JXcqP1zgR5AhsA0agnwAjfB/ow4EH+3HEYV52q

	17 pxN98pUxC+1l2hgVtn8xCS/JGyjK+6dv+lZw3ixJoX2RjVtdJ4/cn318EhhQIDAQAB

	18 -----END PUBLIC KEY-----

	19

	20

	21 $ openssl asn1parse -i < [ALGORITHM]

	22 0:d=0 hl=2 l= 10 cons: SEQUENCE

	23 2:d=1 hl=2 l= 8 prim: OBJECT :ecdsa-with-SHA256

	24 -----BEGIN ALGORITHM-----

	25 MAoGCCqGSM49BAMC

	26 -----END ALGORITHM-----

	27

	28

	29 -----BEGIN DATA-----

	30 MIIB46ADAgECAgkA3l4tFOVii0UwDQYJKoZIhvcNAQELBQAwVjELMAkGA1UEBhMCQVUxEzARBgN

	31 VBAgMClNvbWUtU3RhdGUxITAfBgNVBAoMGEludGVybmV0IFdpZGdpdHMgUHR5IEx0ZDEPMA0GA1

	32 UEAwwGTXkga2V5MB4XDTE1MDcwMjE3MDYzOVoXDTE2MDcwMTE3MDYzOVowVjELMAkGA1UEBhMCQ

	33 VUxEzARBgNVBAgMClNvbWUtU3RhdGUxITAfBgNVBAoMGEludGVybmV0IFdpZGdpdHMgUHR5IEx0

	34 ZDEPMA0GA1UEAwwGTXkga2V5MIGfMA0GCSqGSIb3DQEBAQUAA4GNADCBiQKBgQCqR+B2Mj1irNp

	35 1JnY2zQIQRQPz7ybs6mUjHT3hf5APyaig2u6fBPThqxssgg0YviqIj/70hVK/JXcqP1zgR5AhsA

	36 0agnwAjfB/ow4EH+3HEYV52qpxN98pUxC+1l2hgVtn8xCS/JGyjK+6dv+lZw3ixJoX2RjVtdJ4/

	37 cn318EhhQIDAQABo1AwTjAdBgNVHQ4EFgQUzQBVKTEknyLndWd2HTsBdTKvyikwHwYDVR0jBBgw

	38 FoAUzQBVKTEknyLndWd2HTsBdTKvyikwDAYDVR0TBAUwAwEB/w==

	39 -----END DATA-----

	40

	41

	42 $ openssl asn1parse -i < [SIGNATURE]

	43 0:d=0 hl=3 l= 129 prim: BIT STRING

	44 -----BEGIN SIGNATURE-----

	45 A4GBADrHSmFSJw/Gv7hs5PNzpaJwAri/sitarIZfzN/SjR+n8L8yeTEoiDb1+BkxlFvXvPHTaOK

	46 oO3WlslNNOxh1W5/JkYYGOUkCcyIjnln6qS560imcr3VNjomT/M8M2Iss+rJiKau1TRuaP7H8i6

	47 +Gqf3saGdr8/LnvFAdNQvkalQt

	48 -----END SIGNATURE-----

OLD	NEW