[jira] [Created] (FLINK-4258) Potential null pointer dereference in SavepointCoordinator#onFullyAcknowledgedCheckpoint

[Shang Yuanchun (Jira)]

Ted Yu created FLINK-4258:
-----------------------------

             Summary: Potential null pointer dereference in SavepointCoordinator#onFullyAcknowledgedCheckpoint
                 Key: FLINK-4258
                 URL: https://issues.apache.org/jira/browse/FLINK-4258
             Project: Flink
          Issue Type: Bug
            Reporter: Ted Yu


{code}
    if (promise == null) {
      LOG.info("Pending savepoint with ID " + checkpoint.getCheckpointID() + "  has been " +
          "removed before receiving acknowledgment.");
    }

    // Sanity check
    if (promise.isCompleted()) {
      throw new IllegalStateException("Savepoint promise completed");
{code}
Looks like a return statement is missing in the first if block above.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)