Bug 105983 - Supporting ECDSA (NIST P-256 curve) signatures created by Hungarian citizen eID card (Linux/macOS)
Summary: Supporting ECDSA (NIST P-256 curve) signatures created by Hungarian citizen e...
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: LibreOffice (show other bugs)
Version:
(earliest affected)
5.3.0.3 release
Hardware: All Linux (All)
: medium normal
Assignee: Miklos Vajna
URL:
Whiteboard: target:5.4.0
Keywords:
Depends on:
Blocks: Digital-Signatures
  Show dependency treegraph
 
Reported: 2017-02-13 14:42 UTC by Aron Szabo
Modified: 2017-05-24 07:46 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:


Attachments
LibreOffice document with ECDSA signature (8.07 KB, application/vnd.oasis.opendocument.text)
2017-02-13 14:42 UTC, Aron Szabo
Details
canonicalized SignedInfo (2.06 KB, text/plain)
2017-02-13 14:43 UTC, Aron Szabo
Details
binary SignatureValue (72 bytes, text/plain)
2017-02-13 14:44 UTC, Aron Szabo
Details
base64-encoded public key from X.509 certificate (178 bytes, text/plain)
2017-02-13 14:45 UTC, Aron Szabo
Details
LibreOffice document with ECDSA signature (fixed, 2017-02-24) (8.07 KB, application/vnd.oasis.opendocument.text)
2017-02-24 11:11 UTC, Aron Szabo
Details
LibreOffice document with ECDSA signature (fixed, 2017-03-06) (8.06 KB, application/vnd.oasis.opendocument.text)
2017-03-06 14:34 UTC, Aron Szabo
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Aron Szabo 2017-02-13 14:42:47 UTC
Created attachment 131179 [details]
LibreOffice document with ECDSA signature

I have created a digitally signed LibreOffice document, where ECDSA signature was calculated over the SHA-256 hash of canonicalized SignedInfo XML node. I opened it with LibreOffice 5.3.0.3 but signature verification failed. On low-level I could test the signature with OpenSSL v1.0.1c: result was "Verified OK". (I also performed a negative test to be sure that OpenSSL works well.)

openssl dgst -sha256 -out SignatureValue_verified.txt -keyform PEM -verify Aron_Szabo_end_signer-public_key.pem -signature SignatureValue_binary.txt SignedInfo_to-be-hashed.txt

Could you check ECDSA support on low-level, under XMLSec Library? (I attached test files...)

Relevant modules of my environment:
LibreOffice 5.3.0.3 (x64)
Windows 8.1 Pro (x64)
.NET Framework 4.6.2 (x64)
Comment 1 Aron Szabo 2017-02-13 14:43:57 UTC
Created attachment 131181 [details]
canonicalized SignedInfo
Comment 2 Aron Szabo 2017-02-13 14:44:29 UTC
Created attachment 131182 [details]
binary SignatureValue
Comment 3 Aron Szabo 2017-02-13 14:45:33 UTC
Created attachment 131183 [details]
base64-encoded public key from X.509 certificate
Comment 4 Miklos Vajna 2017-02-13 15:09:54 UTC
dbgutil build prints this error:

warn:xmlsecurity.xmlsec:31557:1:xmlsecurity/source/xmlsec/errorcallback.cxx:48: transforms.c:1571: xmlSecTransformNodeRead() '' 'xmlSecTransformIdListFindByHref' 1 'href=http://www.w3.org/2001/04/xmldsig-more#ecdsa-sha256'
warn:xmlsecurity.xmlsec:31557:1:xmlsecurity/source/xmlsec/errorcallback.cxx:48: transforms.c:704: xmlSecTransformCtxNodeRead() '' 'xmlSecTransformNodeRead' 1 'name=SignatureMethod'
warn:xmlsecurity.xmlsec:31557:1:xmlsecurity/source/xmlsec/errorcallback.cxx:48: xmldsig.c:775: xmlSecDSigCtxProcessSignedInfoNode() '' 'xmlSecTransformCtxNodeRead' 1 'node=SignatureMethod'
warn:xmlsecurity.xmlsec:31557:1:xmlsecurity/source/xmlsec/errorcallback.cxx:48: xmldsig.c:568: xmlSecDSigCtxProcessSignatureNode() '' 'xmlSecDSigCtxProcessSignedInfoNode' 1 ' '
warn:xmlsecurity.xmlsec:31557:1:xmlsecurity/source/xmlsec/errorcallback.cxx:48: xmldsig.c:386: xmlSecDSigCtxVerify() '' 'xmlSecDSigCtxSignatureProcessNode' 1 ' '

Look like the first problem is that xmlsec doesn't recognize the http://www.w3.org/2001/04/xmldsig-more#ecdsa-sha256 string as a valid value for <SignatureMethod Algorithm="..."> when it uses a non-openssl backend (which is a problem for us, as we use the mscrypto/nss backends).

Marking as confirmed.
Comment 5 Aron Szabo 2017-02-24 11:11:03 UTC
Created attachment 131436 [details]
LibreOffice document with ECDSA signature (fixed, 2017-02-24)
Comment 6 Aron Szabo 2017-02-24 11:12:00 UTC
I have uploaded a fixed version of the LibreOffice document with ECDSA signature.

The reason: the unknown name elements of Subject/Issuer fields of the certificates are escaped to 'UNDEF' string instead of their OID value as string by OpenSSL library of PHP. This OID-to-string conversion was fixed in the case of unknown organizationIdentifier (2.5.4.97) name element.

C=HU,L=Budapest,O=NISZ Nemzeti Infokommunikációs Szolgáltató Zrt.,CN=Állampolgári Tanúsítványkiadó - Qualified Citizen CA,2.5.4.97=VATHU-10585560
Comment 7 Miklos Vajna 2017-03-04 21:11:12 UTC
It seems this signature still has a small problem, https://tools.ietf.org/html/rfc4050#section-3.3 says ECDSA signature value should be just the (r,s) pair as-is (with base64 encoding), while the sample applies the ASN1 encoding suggested by https://tools.ietf.org/html/rfc3279#section-2.2.3 -- which is an earlier RFC + applies in general, not in particular to xmldsig.

The result is that xmlsec refuses to verify the signature currently.
Comment 8 Aron Szabo 2017-03-06 14:34:10 UTC
Created attachment 131679 [details]
LibreOffice document with ECDSA signature (fixed, 2017-03-06)
Comment 9 Miklos Vajna 2017-03-06 21:44:37 UTC
Thanks for the update, xmlsec now accepts this on Linux/macOS. Let me use this bug for the xmlsec/nss scenario and once it works, we could have a follow-up bug for the Windows part. Though that won't be too easy, since CryptoAPI doesn't seem to support ECDSA.
Comment 10 Commit Notification 2017-03-08 09:19:27 UTC
Miklos Vajna committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=b5acaa42bb44ad358b0c126af18ea3bdf88d4bd3

tdf#105983 libxmlsec: backport NSS / ECDSA patches

It will be available in 5.4.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 11 Miklos Vajna 2017-05-24 07:46:37 UTC
Follow-up for the Windows part is bug 98545.