HBASE-28868 Add missing permission check for updateRSGroupConfig in branch-2 (#6277)

NihalJain · NihalJain · commit a353ad38d659 · 2024-09-24T22:09:43.000+05:30
Signed-off-by: Duo Zhang <zhangduo@apache.org> (cherry picked from commit cf9c1d5)
diff --git a/hbase-rsgroup/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupAdminEndpoint.java b/hbase-rsgroup/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupAdminEndpoint.java
@@ -447,6 +447,7 @@ public void updateRSGroupConfig(RpcController controller, UpdateRSGroupConfigReq
         if (master.getMasterCoprocessorHost() != null) {
           master.getMasterCoprocessorHost().preUpdateRSGroupConfig(groupName, configuration);
         }
+        checkPermission("updateRSGroupConfig");
         groupAdminServer.updateRSGroupConfig(groupName, configuration);
         if (master.getMasterCoprocessorHost() != null) {
           master.getMasterCoprocessorHost().postUpdateRSGroupConfig(groupName, configuration);

Original file line number	Diff line number	Diff line change
`@@ -447,6 +447,7 @@ public void updateRSGroupConfig(RpcController controller, UpdateRSGroupConfigReq`
`447`	`447`	`if (master.getMasterCoprocessorHost() != null) {`
`448`	`448`	`master.getMasterCoprocessorHost().preUpdateRSGroupConfig(groupName, configuration);`
`449`	`449`	`}`
	`450`	`+ checkPermission("updateRSGroupConfig");`
`450`	`451`	`groupAdminServer.updateRSGroupConfig(groupName, configuration);`
`451`	`452`	`if (master.getMasterCoprocessorHost() != null) {`
`452`	`453`	`master.getMasterCoprocessorHost().postUpdateRSGroupConfig(groupName, configuration);`