From 26f63c331bafd62427ccf660921183e6d6f90a07 Mon Sep 17 00:00:00 2001
From: guluo <lupeng_nwpu@qq.com>
Date: Fri, 10 Nov 2023 00:35:06 +0800
Subject: [PATCH] Meta browser can happen NPE when server or targetServer of
 region is null

---
 .../src/main/resources/hbase-webapps/master/table.jsp      | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/hbase-server/src/main/resources/hbase-webapps/master/table.jsp b/hbase-server/src/main/resources/hbase-webapps/master/table.jsp
index 84718edbc44d..8b88f20301d4 100644
--- a/hbase-server/src/main/resources/hbase-webapps/master/table.jsp
+++ b/hbase-server/src/main/resources/hbase-webapps/master/table.jsp
@@ -504,12 +504,13 @@
       final RegionInfo regionInfo = regionReplicaInfo.getRegionInfo();
       final ServerName serverName = regionReplicaInfo.getServerName();
       final RegionState.State regionState = regionReplicaInfo.getRegionState();
-      final int rsPort = master.getRegionServerInfoPort(serverName);
 
       final long seqNum = regionReplicaInfo.getSeqNum();
 
       final String regionSpanFormat = "<span title=" + HConstants.CATALOG_FAMILY_STR + ":%s>%s</span>";
-      final String targetServerName = regionReplicaInfo.getTargetServerName().toString();
+      final String targetServerName = regionReplicaInfo.getTargetServerName() != null
+        ? regionReplicaInfo.getTargetServerName().toString()
+        : "";
       final Map<String, RegionInfo> mergeRegions = regionReplicaInfo.getMergeRegionInfo();
       final String mergeRegionNames = (mergeRegions == null) ? "" :
         mergeRegions.entrySet().stream()
@@ -527,7 +528,7 @@
           <td title="endKey"><%= endKeyDisplay %></td>
           <td title="replicaId"><%= replicaIdDisplay %></td>
           <td title="regionState"><%= regionStateDisplay %></td>
-          <td title="<%= serverColumnName + "," + startCodeColumnName %>"><%= buildRegionServerLink(serverName, rsPort, regionInfo, regionState) %></td>
+          <td title="<%= serverColumnName + "," + startCodeColumnName %>"><%= serverName != null ? buildRegionServerLink(serverName, master.getRegionServerInfoPort(serverName), regionInfo, regionState) : "" %></td>
           <td title="<%= seqNumColumnName %>"><%= seqNum %></td>
           <td title="<%= serverNameColumnName %>"><%= targetServerName %></td>
           <td><%= mergeRegionNames %></td>