fix(certificate): fix wrong r,s value in ecdsa cert

Blank0120 · Blank0120 · commit c280acc9b14e · 2025-07-30T22:44:50.000+08:00
diff --git a/src/core/operations/ParseX509Certificate.mjs b/src/core/operations/ParseX509Certificate.mjs
@@ -167,8 +167,9 @@ class ParseX509Certificate extends Operation {
         }
 
         if (breakoutSig) { // DSA or ECDSA
-            sigStr = `  r:              ${formatByteStr(r.ASN1HEX.getV(sig, 4), 16, 18)}
-  s:              ${formatByteStr(r.ASN1HEX.getV(sig, 48), 16, 18)}`;
+            const signatureRS = r.KJUR.crypto.ECDSA.parseSigHexInHexRS(sig);
+            sigStr = `  r:              ${formatByteStr(signatureRS.r, 16, 18)}
+  s:              ${formatByteStr(signatureRS.s, 16, 18)}`;
         } else { // RSA or unknown
             sigStr = `  Signature:      ${formatByteStr(sig, 16, 18)}`;
         }

Original file line number	Diff line number	Diff line change
`@@ -167,8 +167,9 @@ class ParseX509Certificate extends Operation {`
`167`	`167`	`}`
`168`	`168`
`169`	`169`	`if (breakoutSig) { // DSA or ECDSA`
`170`		- sigStr = ` r: ${formatByteStr(r.ASN1HEX.getV(sig, 4), 16, 18)}
`171`		- s: ${formatByteStr(r.ASN1HEX.getV(sig, 48), 16, 18)}`;
	`170`	`+ const signatureRS = r.KJUR.crypto.ECDSA.parseSigHexInHexRS(sig);`
	`171`	+ sigStr = ` r: ${formatByteStr(signatureRS.r, 16, 18)}
	`172`	+ s: ${formatByteStr(signatureRS.s, 16, 18)}`;
`172`	`173`	`} else { // RSA or unknown`
`173`	`174`	sigStr = ` Signature: ${formatByteStr(sig, 16, 18)}`;
`174`	`175`	`}`