From 36c3025c974288b8d2322dd9a830d7bff0b155cb Mon Sep 17 00:00:00 2001
From: Michael Stack <stack@apache.org>
Date: Mon, 14 Mar 2011 15:53:31 +0000
Subject: [PATCH] HBASE-3313 Table name isn't checked in
 isTableEnabled/isTableDisabled

git-svn-id: https://svn.apache.org/repos/asf/hbase/trunk@1081431 13f79535-47bb-0310-9956-ffa450edef68
---
 CHANGES.txt                                                  | 2 ++
 src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/CHANGES.txt b/CHANGES.txt
index 8713b854d53..60d682f45a1 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -66,6 +66,8 @@ Release 0.91.0 - Unreleased
                not exit
    HBASE-3626  Update instructions in thrift demo files (Moaz Reyad via Stack)
    HBASE-3538  Column families allow to have slashes in name (Ian Knome via Stack)
+   HBASE-3313  Table name isn't checked in isTableEnabled/isTableDisabled
+               (Ted Yu via Stack)
 
   IMPROVEMENTS
    HBASE-3290  Max Compaction Size (Nicolas Spiegelberg via Stack)  
diff --git a/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java b/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java
index 82993a7b301..69b14a54ed9 100644
--- a/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java
+++ b/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java
@@ -590,6 +590,7 @@ public class HBaseAdmin implements Abortable {
    * @throws IOException if a remote or network exception occurs
    */
   public boolean isTableEnabled(byte[] tableName) throws IOException {
+    HTableDescriptor.isLegalTableName(tableName);
     return connection.isTableEnabled(tableName);
   }
 
@@ -608,6 +609,7 @@ public class HBaseAdmin implements Abortable {
    * @throws IOException if a remote or network exception occurs
    */
   public boolean isTableDisabled(byte[] tableName) throws IOException {
+    HTableDescriptor.isLegalTableName(tableName);
     return connection.isTableDisabled(tableName);
   }