From 961ab7390b5c938ae5aa77254504ca4eff955c6f Mon Sep 17 00:00:00 2001
From: Istvan Toth <stoty@apache.org>
Date: Tue, 12 Sep 2023 15:24:47 +0200
Subject: [PATCH] HBASE-28076 NPE on initialization error in
 RecoveredReplicationSourceShipper

Change-Id: I98d669163842287905cd384e155225269fad1744
---
 .../regionserver/RecoveredReplicationSourceShipper.java     | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/RecoveredReplicationSourceShipper.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/RecoveredReplicationSourceShipper.java
index 471ca68f6599..b01df7db8493 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/RecoveredReplicationSourceShipper.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/RecoveredReplicationSourceShipper.java
@@ -97,8 +97,10 @@ private void terminate(String reason, Exception cause) {
         "Closing worker for wal group " + this.walGroupId + " because an error occurred: " + reason,
         cause);
     }
-    entryReader.interrupt();
-    Threads.shutdown(entryReader, sleepForRetries);
+    if (entryReader != null) {
+      entryReader.interrupt();
+      Threads.shutdown(entryReader, sleepForRetries);
+    }
     this.interrupt();
     Threads.shutdown(this, sleepForRetries);
     LOG.info("ReplicationSourceWorker {} terminated", this.getName());