Implement recoverSessionKey(pkesk, encData) in AbstractPublicKeyDataD…

…ecryptorFactory
pgpainless · Sep 10, 2024 · 398c845 · 398c845
1 parent 8ffaaba
commit 398c845
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 25 deletions.
diff --git a/...rc/main/java/org/bouncycastle/openpgp/operator/AbstractPublicKeyDataDecryptorFactory.java b/...rc/main/java/org/bouncycastle/openpgp/operator/AbstractPublicKeyDataDecryptorFactory.java
@@ -13,6 +13,15 @@ public abstract class AbstractPublicKeyDataDecryptorFactory
         implements PublicKeyDataDecryptorFactory
 {
 
+    @Override
+    public final byte[] recoverSessionData(PublicKeyEncSessionPacket pkesk, InputStreamPacket encData)
+            throws PGPException
+    {
+        byte[] sessionData = recoverSessionData(pkesk.getAlgorithm(), pkesk.getEncSessionKey(), pkesk.getVersion());
+        return prependSKAlgorithmToSessionData(pkesk, encData, sessionData);
+    }
+
+
     protected byte[] prependSKAlgorithmToSessionData(PublicKeyEncSessionPacket pkesk,
                                                    InputStreamPacket encData,
                                                    byte[] decryptedSessionData)

diff --git a/pg/src/main/java/org/bouncycastle/openpgp/operator/bc/BcPublicKeyDataDecryptorFactory.java b/pg/src/main/java/org/bouncycastle/openpgp/operator/bc/BcPublicKeyDataDecryptorFactory.java
@@ -56,13 +56,6 @@ public BcPublicKeyDataDecryptorFactory(PGPPrivateKey pgpPrivKey)
         this.pgpPrivKey = pgpPrivKey;
     }
 
-    public byte[] recoverSessionData(PublicKeyEncSessionPacket pkesk, InputStreamPacket encData)
-        throws PGPException
-    {
-        byte[] sessionData = recoverSessionData(pkesk.getAlgorithm(), pkesk.getEncSessionKey(), pkesk.getVersion());
-        return prependSKAlgorithmToSessionData(pkesk, encData, sessionData);
-    }
-
     @Override
     public byte[] recoverSessionData(int keyAlgorithm, byte[][] secKeyData, int pkeskVersion)
         throws PGPException

diff --git a/...ava/org/bouncycastle/openpgp/operator/jcajce/JcePublicKeyDataDecryptorFactoryBuilder.java b/...ava/org/bouncycastle/openpgp/operator/jcajce/JcePublicKeyDataDecryptorFactoryBuilder.java
@@ -24,9 +24,7 @@
 import org.bouncycastle.asn1.x9.X9ECParametersHolder;
 import org.bouncycastle.bcpg.AEADEncDataPacket;
 import org.bouncycastle.bcpg.ECDHPublicBCPGKey;
-import org.bouncycastle.bcpg.InputStreamPacket;
 import org.bouncycastle.bcpg.PublicKeyAlgorithmTags;
-import org.bouncycastle.bcpg.PublicKeyEncSessionPacket;
 import org.bouncycastle.bcpg.PublicKeyPacket;
 import org.bouncycastle.bcpg.SymmetricEncIntegrityPacket;
 import org.bouncycastle.bcpg.SymmetricKeyAlgorithmTags;
@@ -134,14 +132,6 @@ public PublicKeyDataDecryptorFactory build(final PrivateKey privKey)
         {
             final int expectedPayLoadSize = getExpectedPayloadSize(privKey);
 
-            @Override
-            public byte[] recoverSessionData(PublicKeyEncSessionPacket pkesk, InputStreamPacket encData)
-                    throws PGPException
-            {
-                byte[] sessionData = recoverSessionData(pkesk.getAlgorithm(), pkesk.getEncSessionKey(), pkesk.getVersion());
-                return prependSKAlgorithmToSessionData(pkesk, encData, sessionData);
-            }
-
             @Override
             public byte[] recoverSessionData(int keyAlgorithm, byte[][] secKeyData, int pkeskVersion)
                 throws PGPException
@@ -183,14 +173,6 @@ public PublicKeyDataDecryptorFactory build(final PGPPrivateKey privKey)
     {
         return new AbstractPublicKeyDataDecryptorFactory()
         {
-            @Override
-            public byte[] recoverSessionData(PublicKeyEncSessionPacket pkesk, InputStreamPacket encData)
-                    throws PGPException
-            {
-                byte[] sessionData = recoverSessionData(pkesk.getAlgorithm(), pkesk.getEncSessionKey(), pkesk.getVersion());
-                return prependSKAlgorithmToSessionData(pkesk, encData, sessionData);
-            }
-
             @Override
             public byte[] recoverSessionData(int keyAlgorithm, byte[][] secKeyData, int pkeskVersion)
                 throws PGPException