You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@thrift.apache.org by ro...@apache.org on 2014/07/26 00:45:52 UTC
git commit: THRIFT-2122 NPE in TSSLTransportFactory.createClient when
no truststore password is passed in custom SSL params
Repository: thrift
Updated Branches:
refs/heads/master e3f0a4234 -> 54c0d2e8e
THRIFT-2122 NPE in TSSLTransportFactory.createClient when no truststore password is passed in custom SSL params
Client: Java
Patch: Venki Korukanti
Project: http://git-wip-us.apache.org/repos/asf/thrift/repo
Commit: http://git-wip-us.apache.org/repos/asf/thrift/commit/54c0d2e8
Tree: http://git-wip-us.apache.org/repos/asf/thrift/tree/54c0d2e8
Diff: http://git-wip-us.apache.org/repos/asf/thrift/diff/54c0d2e8
Branch: refs/heads/master
Commit: 54c0d2e8e71678f3f77339f4b3a3a640c29aeaa4
Parents: e3f0a42
Author: Roger Meier <ro...@apache.org>
Authored: Sat Jul 26 00:37:44 2014 +0200
Committer: Roger Meier <ro...@apache.org>
Committed: Sat Jul 26 00:37:44 2014 +0200
----------------------------------------------------------------------
.../thrift/transport/TSSLTransportFactory.java | 3 +-
.../TestTSSLTransportFactoryCustomClient1.java | 35 ++++++++++++++++++++
.../TestTSSLTransportFactoryCustomClient2.java | 34 +++++++++++++++++++
3 files changed, 71 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/thrift/blob/54c0d2e8/lib/java/src/org/apache/thrift/transport/TSSLTransportFactory.java
----------------------------------------------------------------------
diff --git a/lib/java/src/org/apache/thrift/transport/TSSLTransportFactory.java b/lib/java/src/org/apache/thrift/transport/TSSLTransportFactory.java
index 16d2fd1..e830bb9 100755
--- a/lib/java/src/org/apache/thrift/transport/TSSLTransportFactory.java
+++ b/lib/java/src/org/apache/thrift/transport/TSSLTransportFactory.java
@@ -183,7 +183,8 @@ public class TSSLTransportFactory {
tmf = TrustManagerFactory.getInstance(params.trustManagerType);
KeyStore ts = KeyStore.getInstance(params.trustStoreType);
fin = new FileInputStream(params.trustStore);
- ts.load(fin, params.trustPass.toCharArray());
+ ts.load(fin,
+ (params.trustPass != null ? params.trustPass.toCharArray() : null));
tmf.init(ts);
}
http://git-wip-us.apache.org/repos/asf/thrift/blob/54c0d2e8/lib/java/test/org/apache/thrift/transport/TestTSSLTransportFactoryCustomClient1.java
----------------------------------------------------------------------
diff --git a/lib/java/test/org/apache/thrift/transport/TestTSSLTransportFactoryCustomClient1.java b/lib/java/test/org/apache/thrift/transport/TestTSSLTransportFactoryCustomClient1.java
new file mode 100644
index 0000000..da1659f
--- /dev/null
+++ b/lib/java/test/org/apache/thrift/transport/TestTSSLTransportFactoryCustomClient1.java
@@ -0,0 +1,35 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+package org.apache.thrift.transport;
+
+public class TestTSSLTransportFactoryCustomClient1 extends TestTSSLTransportFactory {
+
+ @Override
+ public TTransport getClientTransport(TTransport underlyingTransport)
+ throws Exception {
+ TSSLTransportFactory.TSSLTransportParameters params = new
+ TSSLTransportFactory.TSSLTransportParameters();
+
+ params.setTrustStore(System.getProperty("javax.net.ssl.trustStore"),
+ System.getProperty("javax.net.ssl.trustStorePassword"));
+
+ return TSSLTransportFactory.getClientSocket(HOST, PORT, 0/*timeout*/, params);
+ }
+}
http://git-wip-us.apache.org/repos/asf/thrift/blob/54c0d2e8/lib/java/test/org/apache/thrift/transport/TestTSSLTransportFactoryCustomClient2.java
----------------------------------------------------------------------
diff --git a/lib/java/test/org/apache/thrift/transport/TestTSSLTransportFactoryCustomClient2.java b/lib/java/test/org/apache/thrift/transport/TestTSSLTransportFactoryCustomClient2.java
new file mode 100644
index 0000000..eaed460
--- /dev/null
+++ b/lib/java/test/org/apache/thrift/transport/TestTSSLTransportFactoryCustomClient2.java
@@ -0,0 +1,34 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+package org.apache.thrift.transport;
+
+public class TestTSSLTransportFactoryCustomClient2 extends TestTSSLTransportFactory {
+
+ @Override
+ public TTransport getClientTransport(TTransport underlyingTransport)
+ throws Exception {
+ TSSLTransportFactory.TSSLTransportParameters params = new
+ TSSLTransportFactory.TSSLTransportParameters();
+
+ params.setTrustStore(System.getProperty("javax.net.ssl.trustStore"), null);
+
+ return TSSLTransportFactory.getClientSocket(HOST, PORT, 0/*timeout*/, params);
+ }
+}