Repository: incubator-geode
Updated Branches:
  refs/heads/feature/GEODE-773-2 3504a05ff -> 25d5db399


Fixing an NPE that showed up in an integration test.


Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/fd88a6a9
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/fd88a6a9
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/fd88a6a9

Branch: refs/heads/feature/GEODE-773-2
Commit: fd88a6a9a55a4ac06cb083698313bcb7a09973ea
Parents: c8c26e6
Author: Bruce Schuchardt <bschucha...@pivotal.io>
Authored: Mon Feb 8 15:54:28 2016 -0800
Committer: Bruce Schuchardt <bschucha...@pivotal.io>
Committed: Mon Feb 8 15:54:28 2016 -0800

----------------------------------------------------------------------
 .../distributed/internal/InternalDistributedSystem.java     | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/fd88a6a9/gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/InternalDistributedSystem.java
----------------------------------------------------------------------
diff --git 
a/gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/InternalDistributedSystem.java
 
b/gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/InternalDistributedSystem.java
index 5762463..276efec 100644
--- 
a/gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/InternalDistributedSystem.java
+++ 
b/gemfire-core/src/main/java/com/gemstone/gemfire/distributed/internal/InternalDistributedSystem.java
@@ -778,7 +778,8 @@ public class InternalDistributedSystem
    * @since 5.7
    */
   private void endInitLocator() throws IOException {
-    if (startedLocator != null) {
+    InternalLocator loc = this.startedLocator;
+    if (loc != null) {
       String locatorString = this.originalConfig.getStartLocator();
 //      DistributionLocatorId locId = new DistributionLocatorId(locatorString);
       boolean finished = false;
@@ -787,14 +788,14 @@ public class InternalDistributedSystem
         // start server location services in order to be able to log 
information
         // about the use of cache servers
         //      if(locId.isServerLocator()) {
-        this.startedLocator.startServerLocation(this);
+        loc.startServerLocation(this);
         //      }
       
-        this.startedLocator.endStartLocator(this);
+        loc.endStartLocator(this);
         finished = true;
       } finally {
         if (!finished) {
-          this.startedLocator.stop();
+          loc.stop();
         }
       }
     }

Reply via email to