diff --git a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.HDFS-1623.txt b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.HDFS-1623.txt index 91d58c04a2..21d9b7d9db 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.HDFS-1623.txt +++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.HDFS-1623.txt @@ -7,3 +7,5 @@ branch is merged. HDFS-2179. Add fencing framework and mechanisms for NameNode HA. (todd) HDFS-1974. Introduce active and standy states to the namenode. (suresh) + +HDFS-2407. getServerDefaults and getStats don't check operation category (atm) diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NameNodeRpcServer.java b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NameNodeRpcServer.java index 5b0d7a79cf..d79614f7d4 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NameNodeRpcServer.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NameNodeRpcServer.java @@ -319,6 +319,7 @@ class NameNodeRpcServer implements NamenodeProtocols { @Override // ClientProtocol public FsServerDefaults getServerDefaults() throws IOException { + nn.checkOperation(OperationCategory.READ); return namesystem.getServerDefaults(); } @@ -634,8 +635,9 @@ class NameNodeRpcServer implements NamenodeProtocols { return namesystem.getFileInfo(src, false); } - @Override - public long[] getStats() { + @Override // ClientProtocol + public long[] getStats() throws IOException { + nn.checkOperation(OperationCategory.READ); return namesystem.getStats(); }