[ 
https://issues.apache.org/jira/browse/HBASE-16284?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15396162#comment-15396162
 ] 

Jerry He commented on HBASE-16284:
----------------------------------

The patch looks good.

{code}
+    } catch (IOException e) {
+      LOG.error("Exception occurred in HMaster.shutdown()", e);
+      if (e instanceof AccessDeniedException) {
+        throw new ServiceException(e);
+      }
+    }
{code}

Could we just throw ServiceException for all IOException ?


> Unauthorized client can shutdown the cluster
> --------------------------------------------
>
>                 Key: HBASE-16284
>                 URL: https://issues.apache.org/jira/browse/HBASE-16284
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Deokwoo Han
>         Attachments: HBASE-16284.patch
>
>
> An unauthorized client can shutdown the cluster as {{AccessDeniedException}} 
> is ignored during {{Admin.stopMaster}} and {{Admin.shutdown}}.



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

Reply via email to