Return-Path: X-Original-To: apmail-ignite-commits-archive@minotaur.apache.org Delivered-To: apmail-ignite-commits-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 2CADF17F6A for ; Tue, 2 Jun 2015 12:59:35 +0000 (UTC) Received: (qmail 88048 invoked by uid 500); 2 Jun 2015 12:59:35 -0000 Delivered-To: apmail-ignite-commits-archive@ignite.apache.org Received: (qmail 88012 invoked by uid 500); 2 Jun 2015 12:59:35 -0000 Mailing-List: contact commits-help@ignite.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@ignite.incubator.apache.org Delivered-To: mailing list commits@ignite.incubator.apache.org Received: (qmail 87962 invoked by uid 99); 2 Jun 2015 12:59:35 -0000 Received: from Unknown (HELO spamd1-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 02 Jun 2015 12:59:35 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd1-us-west.apache.org (ASF Mail Server at spamd1-us-west.apache.org) with ESMTP id 9ED65CABCC for ; Tue, 2 Jun 2015 12:59:34 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd1-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 1.791 X-Spam-Level: * X-Spam-Status: No, score=1.791 tagged_above=-999 required=6.31 tests=[KAM_ASCII_DIVIDERS=0.8, KAM_LAZY_DOMAIN_SECURITY=1, T_RP_MATCHES_RCVD=-0.01, URIBL_BLOCKED=0.001] autolearn=disabled Received: from mx1-eu-west.apache.org ([10.40.0.8]) by localhost (spamd1-us-west.apache.org [10.40.0.7]) (amavisd-new, port 10024) with ESMTP id SFEwzCUGNT6g for ; Tue, 2 Jun 2015 12:59:19 +0000 (UTC) Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx1-eu-west.apache.org (ASF Mail Server at mx1-eu-west.apache.org) with SMTP id DFEFC253F9 for ; Tue, 2 Jun 2015 12:59:13 +0000 (UTC) Received: (qmail 84406 invoked by uid 99); 2 Jun 2015 12:59:12 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 02 Jun 2015 12:59:12 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 10DE5E0AFD; Tue, 2 Jun 2015 12:59:12 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: yzhdanov@apache.org To: commits@ignite.incubator.apache.org Date: Tue, 02 Jun 2015 12:59:18 -0000 Message-Id: In-Reply-To: <2812d8a3579948a8acaf1dc25381e735@git.apache.org> References: <2812d8a3579948a8acaf1dc25381e735@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: [08/50] [abbrv] incubator-ignite git commit: # IGNITE-943 Bug fix: NPE in setAuthenticator # IGNITE-943 Bug fix: NPE in setAuthenticator Project: http://git-wip-us.apache.org/repos/asf/incubator-ignite/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ignite/commit/6b1265c2 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ignite/tree/6b1265c2 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ignite/diff/6b1265c2 Branch: refs/heads/ignite-745 Commit: 6b1265c2b103b4131b3ffc6deecd55d440fbe9b0 Parents: 2bc0795 Author: sevdokimov Authored: Thu May 28 22:06:29 2015 +0300 Committer: sevdokimov Committed: Thu May 28 22:06:29 2015 +0300 ---------------------------------------------------------------------- .../ignite/spi/discovery/tcp/ClientImpl.java | 5 ----- .../ignite/spi/discovery/tcp/ServerImpl.java | 20 ++++++-------------- .../spi/discovery/tcp/TcpDiscoveryImpl.java | 5 ----- .../spi/discovery/tcp/TcpDiscoverySpi.java | 5 ++++- 4 files changed, 10 insertions(+), 25 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/6b1265c2/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/ClientImpl.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/ClientImpl.java b/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/ClientImpl.java index 2171085..e672d64 100644 --- a/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/ClientImpl.java +++ b/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/ClientImpl.java @@ -319,11 +319,6 @@ class ClientImpl extends TcpDiscoveryImpl { } /** {@inheritDoc} */ - @Override public void setAuthenticator(DiscoverySpiNodeAuthenticator auth) { - // No-op. - } - - /** {@inheritDoc} */ @Override public void sendCustomEvent(DiscoverySpiCustomMessage evt) { if (segmented) throw new IgniteException("Failed to send custom message: client is disconnected"); http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/6b1265c2/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/ServerImpl.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/ServerImpl.java b/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/ServerImpl.java index b5c9519..57c13d6 100644 --- a/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/ServerImpl.java +++ b/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/ServerImpl.java @@ -115,9 +115,6 @@ class ServerImpl extends TcpDiscoveryImpl { /** Response on join request from coordinator (in case of duplicate ID or auth failure). */ private final GridTuple joinRes = F.t1(); - /** Node authenticator. */ - private DiscoverySpiNodeAuthenticator nodeAuth; - /** Mutex. */ private final Object mux = new Object(); @@ -631,11 +628,6 @@ class ServerImpl extends TcpDiscoveryImpl { } /** {@inheritDoc} */ - @Override public void setAuthenticator(DiscoverySpiNodeAuthenticator nodeAuth) { - this.nodeAuth = nodeAuth; - } - - /** {@inheritDoc} */ @Override public void sendCustomEvent(DiscoverySpiCustomMessage evt) { try { msgWorker.addMessage(new TcpDiscoveryCustomEventMessage(getLocalNodeId(), evt, spi.marsh.marshal(evt))); @@ -680,10 +672,10 @@ class ServerImpl extends TcpDiscoveryImpl { if (log.isDebugEnabled()) log.debug("Join request message has not been sent (local node is the first in the topology)."); - if (nodeAuth != null) { + if (spi.nodeAuth != null) { // Authenticate local node. try { - SecurityContext subj = nodeAuth.authenticateNode(locNode, locCred); + SecurityContext subj = spi.nodeAuth.authenticateNode(locNode, locCred); if (subj == null) throw new IgniteSpiException("Authentication failed for local node: " + locNode.id()); @@ -2469,12 +2461,12 @@ class ServerImpl extends TcpDiscoveryImpl { return; } - if (nodeAuth != null) { + if (spi.nodeAuth != null) { // Authenticate node first. try { SecurityCredentials cred = unmarshalCredentials(node); - SecurityContext subj = nodeAuth.authenticateNode(node, cred); + SecurityContext subj = spi.nodeAuth.authenticateNode(node, cred); if (subj == null) { // Node has not pass authentication. @@ -2843,7 +2835,7 @@ class ServerImpl extends TcpDiscoveryImpl { return; } - if (!isLocalNodeCoordinator() && nodeAuth != null && nodeAuth.isGlobalNodeAuthentication()) { + if (!isLocalNodeCoordinator() && spi.nodeAuth != null && spi.nodeAuth.isGlobalNodeAuthentication()) { boolean authFailed = true; try { @@ -2861,7 +2853,7 @@ class ServerImpl extends TcpDiscoveryImpl { authFailed = false; } else { - SecurityContext subj = nodeAuth.authenticateNode(node, cred); + SecurityContext subj = spi.nodeAuth.authenticateNode(node, cred); SecurityContext coordSubj = spi.ignite().configuration().getMarshaller().unmarshal( node.attribute(IgniteNodeAttributes.ATTR_SECURITY_SUBJECT), http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/6b1265c2/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/TcpDiscoveryImpl.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/TcpDiscoveryImpl.java b/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/TcpDiscoveryImpl.java index 4836911..f285279 100644 --- a/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/TcpDiscoveryImpl.java +++ b/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/TcpDiscoveryImpl.java @@ -113,11 +113,6 @@ abstract class TcpDiscoveryImpl { public abstract void disconnect() throws IgniteSpiException; /** - * @param auth Auth. - */ - public abstract void setAuthenticator(DiscoverySpiNodeAuthenticator auth); - - /** * @param msg Message. */ public abstract void sendCustomEvent(DiscoverySpiCustomMessage msg) throws IgniteException; http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/6b1265c2/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/TcpDiscoverySpi.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/TcpDiscoverySpi.java b/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/TcpDiscoverySpi.java index 64e6001..1b8424c 100644 --- a/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/TcpDiscoverySpi.java +++ b/modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/TcpDiscoverySpi.java @@ -294,6 +294,9 @@ public class TcpDiscoverySpi extends IgniteSpiAdapter implements DiscoverySpi, T @SuppressWarnings({"FieldAccessedSynchronizedAndUnsynchronized"}) protected long ipFinderCleanFreq = DFLT_IP_FINDER_CLEAN_FREQ; + /** Node authenticator. */ + protected DiscoverySpiNodeAuthenticator nodeAuth; + /** Context initialization latch. */ @GridToStringExclude private final CountDownLatch ctxInitLatch = new CountDownLatch(1); @@ -353,7 +356,7 @@ public class TcpDiscoverySpi extends IgniteSpiAdapter implements DiscoverySpi, T /** {@inheritDoc} */ @Override public void setAuthenticator(DiscoverySpiNodeAuthenticator auth) { - impl.setAuthenticator(auth); + nodeAuth = auth; } /** {@inheritDoc} */