From 2b53bdf88248b5842f12a14eef7f63c959d06c6f Mon Sep 17 00:00:00 2001
From: mores <stephen.more@gmail.com>
Date: Mon, 6 May 2013 13:46:47 -0400
Subject: [PATCH] getSocketFactory() should only be called on a sslContext that
 has been initialized.

---
 .../org/jasig/cas/client/ssl/HttpsURLConnectionFactory.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cas-client-core/src/main/java/org/jasig/cas/client/ssl/HttpsURLConnectionFactory.java b/cas-client-core/src/main/java/org/jasig/cas/client/ssl/HttpsURLConnectionFactory.java
index 2475e5a..298069a 100644
--- a/cas-client-core/src/main/java/org/jasig/cas/client/ssl/HttpsURLConnectionFactory.java
+++ b/cas-client-core/src/main/java/org/jasig/cas/client/ssl/HttpsURLConnectionFactory.java
@@ -118,11 +118,11 @@ public final class HttpsURLConnectionFactory implements HttpURLConnectionFactory
                         final KeyManagerFactory keyManager = KeyManagerFactory.getInstance(this.sslConfiguration.getProperty("keyManagerType", "SunX509"));
                         keyManager.init(keyStore, this.sslConfiguration.getProperty("certificatePassword").toCharArray());
                         sslContext.init(keyManager.getKeyManagers(), null, null);
+                        return sslContext.getSocketFactory();
                     }
                 }
             }
 
-            return sslContext.getSocketFactory();
         } catch (final Exception e) {
             LOGGER.error(e.getMessage(), e);
         } finally {