Bruce J Schuchardt created GEODE-7592:
-----------------------------------------

             Summary: possible NPE in SystemManagementService
                 Key: GEODE-7592
                 URL: https://issues.apache.org/jira/browse/GEODE-7592
             Project: Geode
          Issue Type: Improvement
          Components: management
            Reporter: Bruce J Schuchardt


LGTM is reporting a possible NPE in this code:
{code:java}
if (federatingManager != null && federatingManager.isRunning()) {
  throw new AlreadyRunningException(
      "Manager is already running");
}

boolean needsToBeStarted = false;
if (!isManagerCreated()) {
  createManager();
  needsToBeStarted = true;
} else if (!federatingManager.isRunning()) {
  needsToBeStarted = true;
} {code}
 

There is a null check in the first "if" statement but no check in the 
subsequent "else if" statement.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to