Blame SOURCES/pki-core-tps-externalReg-regression.patch

120910
From 47fccfaa32bf327c56c5789f373e4d18c714556f Mon Sep 17 00:00:00 2001
120910
From: Jack Magne <jmagne@redhat.com>
120910
Date: Fri, 10 Nov 2017 15:55:36 -0800
120910
Subject: [PATCH] ReFix for  #2824 TPS new configuration to allow the protocol
120910
 of the to determine applet loaded.
120910
120910
The problem discovered was that in only the external registration case, there was a problem obtaining the protocol
120910
information for the token being enrolled. This simple fix makes sure the protocol info is obtained correctly for external
120910
reg and non external reg enrollment cases.
120910
120910
Change-Id: Iccd40adbdafd5e94e04cbb8c391bd2706e483a1f
120910
(cherry picked from commit e48374cd8a744fad5a03f64e8685ec3b3c465553)
120910
(cherry picked from commit 3718b369598c3ecfdb6733c25b08e4b51042423f)
120910
---
120910
 .../src/org/dogtagpki/server/tps/processor/TPSProcessor.java   | 10 ++++++----
120910
 1 file changed, 6 insertions(+), 4 deletions(-)
120910
120910
diff --git a/base/tps/src/org/dogtagpki/server/tps/processor/TPSProcessor.java b/base/tps/src/org/dogtagpki/server/tps/processor/TPSProcessor.java
120910
index 4ec277e..03d6532 100644
120910
--- a/base/tps/src/org/dogtagpki/server/tps/processor/TPSProcessor.java
120910
+++ b/base/tps/src/org/dogtagpki/server/tps/processor/TPSProcessor.java
120910
@@ -33,8 +33,6 @@ import java.util.List;
120910
 import java.util.Map;
120910
 import java.util.Set;
120910
 
120910
-import netscape.security.x509.RevocationReason;
120910
-
120910
 import org.dogtagpki.server.tps.TPSSession;
120910
 import org.dogtagpki.server.tps.TPSSubsystem;
120910
 import org.dogtagpki.server.tps.authentication.AuthUIParameter;
120910
@@ -102,6 +100,8 @@ import com.netscape.cms.servlet.tks.SecureChannelProtocol;
120910
 import com.netscape.cmsutil.crypto.CryptoUtil;
120910
 import com.netscape.symkey.SessionKey;
120910
 
120910
+import netscape.security.x509.RevocationReason;
120910
+
120910
 public class TPSProcessor {
120910
 
120910
     public static final int RESULT_NO_ERROR = 0;
120910
@@ -556,8 +556,10 @@ public class TPSProcessor {
120910
             CMS.debug("TPSProcessor.setupSecureChannel: obtained randomData");
120910
         }
120910
 
120910
-        // We already do this when checking for applet upgrade earlier.
120910
-        //acquireChannelPlatformAndProtocolInfo();
120910
+        // Do this on behalf of external reg, which needs it
120910
+        // If already called, the routine will return anyway.
120910
+
120910
+        acquireChannelPlatformAndProtocolInfo();
120910
 
120910
         TPSBuffer initUpdateResp = initializeUpdate(keyVersion, keyIndex, randomData);
120910
 
120910
-- 
120910
1.8.3.1
120910