GHSA-p8c3-7rj8-q963
Suggest an improvement- Source
- https://github.com/advisories/GHSA-p8c3-7rj8-q963
- Import Source
- https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2020/06/GHSA-p8c3-7rj8-q963/GHSA-p8c3-7rj8-q963.json
- JSON Data
- https://api.osv.dev/v1/vulns/GHSA-p8c3-7rj8-q963
- Aliases
- Published
- 2020-06-26T16:54:15Z
- Modified
- 2023-11-08T04:02:28.073703Z
- Severity
-
- 7.5 (High) CVSS_V3 - CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:H/A:N CVSS Calculator
- Summary
- ECDSA signature validation vulnerability by accepting wrong ASN.1 encoding in jsrsasign
- Details
-
Impact
Jsrsasign supports ECDSA signature validation which signature value is represented by ASN.1 DER encoding. This vulnerablity may accept a wrong ASN.1 DER encoded ECDSA signature such as:
- wrong multi-byte ASN.1 length of TLV (ex. 0x820045 even though 0x45 is correct)
- prepending zeros with ASN.1 INTEGER value (ex. 0x00000123 even though 0x0123 is correct)
- appending zeros to signature of ASN.1 TLV (ex. 0x3082....1fbc000000 even though 0x3082....1fbc, appending zeros are ignored.)
This vulnerability was fixed by strict ASN.1 DER checking.
Here is an assessment of this vulnerability:
- If you are not use ECDSA signature validation, this vulnerability is not affected.
- Not ASN.1 format signature like just concatenation of R and S value is not affected such as Bitcoin.
- This vulnerability is affected to all ECC curve parameters.
- Risk to accept a forged or crafted message to be signed is low.
- Risk to raise memory corruption is low since jsrsasign uses BigInteger class.
- ECDSA signatures semantically the same to valid one may be accepted as valid. There are many malleable variants.
As discussed here, there is no standards like X9.62 which requires ASN.1 DER. So ASN.1 BER can be applied to ECDSA however most of implementations like OpenSSL do strict ASN.1 DER checking.
Patches
Users using ECDSA signature validation should upgrade to 8.0.19.
Workarounds
Do strict ASN.1 DER checking for ASN.1 encoded ECDSA signature value.
References
https://nvd.nist.gov/vuln/detail/CVE-2020-14966 https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2020-14966 https://vuldb.com/?id.157123 https://github.com/kjur/jsrsasign/issues/437 https://kjur.github.io/jsrsasign/api/symbols/KJUR.crypto.ECDSA.html https://kjur.github.io/jsrsasign/api/symbols/ASN1HEX.html#.checkStrictDER https://www.itu.int/rec/T-REC-X.690
- Database specific
{ "nvd_published_at": "2020-06-22T12:15:00Z", "cwe_ids": [ "CWE-347" ], "severity": "HIGH", "github_reviewed": true, "github_reviewed_at": "2020-06-26T16:54:00Z" }- References
-
- https://github.com/kjur/jsrsasign/security/advisories/GHSA-p8c3-7rj8-q963
- https://nvd.nist.gov/vuln/detail/CVE-2020-14966
- https://github.com/kjur/jsrsasign/issues/437
- https://github.com/kjur/jsrsasign/commit/6087412d072a57074d3c4c1b40bdde0460d53a7f
- https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2020-14966
- https://github.com/kjur/jsrsasign
- https://github.com/kjur/jsrsasign/releases/tag/8.0.17
- https://github.com/kjur/jsrsasign/releases/tag/8.0.18
- https://kjur.github.io/jsrsasign
- https://kjur.github.io/jsrsasign/api/symbols/ASN1HEX.html#.checkStrictDER
- https://kjur.github.io/jsrsasign/api/symbols/KJUR.crypto.ECDSA.html
- https://security.netapp.com/advisory/ntap-20200724-0001
- https://vuldb.com/?id.157123
- https://www.npmjs.com/package/jsrsasign
Affected packages
npm / jsrsasign
Package
- Name
- jsrsasign
- View open source insights on deps.dev
- Purl
- pkg:npm/jsrsasign