TKSS-1018: Native SM2Signature should not reset NativeSMSignature ins…

…tance
Tencent · Dec 27, 2024 · 557a837 · 557a837
1 parent 279ee5c
commit 557a837
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/kona-crypto/src/main/java/com/tencent/kona/crypto/provider/nativeImpl/SM2Signature.java b/kona-crypto/src/main/java/com/tencent/kona/crypto/provider/nativeImpl/SM2Signature.java
@@ -160,7 +160,7 @@ protected byte[] engineSign() throws SignatureException {
         } catch (BadPaddingException e) {
             throw new SignatureException(e);
         } finally {
-            reset();
+            buffer.reset();
         }
     }
 
@@ -184,12 +184,15 @@ protected boolean engineVerify(byte[] sigBytes) throws SignatureException {
         } catch (BadPaddingException e) {
             throw new SignatureException(e);
         } finally {
-            reset();
+            buffer.reset();
         }
     }
 
     private void reset() {
-        sm2 = null;
+        privateKey = null;
+        publicKey = null;
+        id = null;
+
         buffer.reset();
     }
 }