You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by aw...@apache.org on 2015/07/30 03:27:39 UTC
[09/14] hadoop git commit: HADOOP-10945. 4-digit octal umask
permissions throws a parse error. Contributed by Chang Li
HADOOP-10945. 4-digit octal umask permissions throws a parse error. Contributed by Chang Li
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/6f0a3572
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/6f0a3572
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/6f0a3572
Branch: refs/heads/HADOOP-12111
Commit: 6f0a35724f0da80146dbae4b6f6c341e1d3101f5
Parents: 2b2bd92
Author: Jason Lowe <jl...@apache.org>
Authored: Wed Jul 29 15:25:10 2015 +0000
Committer: Jason Lowe <jl...@apache.org>
Committed: Wed Jul 29 15:25:10 2015 +0000
----------------------------------------------------------------------
hadoop-common-project/hadoop-common/CHANGES.txt | 3 +++
.../hadoop/fs/permission/UmaskParser.java | 2 +-
.../apache/hadoop/security/TestPermission.java | 26 ++++++++++++++++++++
3 files changed, 30 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hadoop/blob/6f0a3572/hadoop-common-project/hadoop-common/CHANGES.txt
----------------------------------------------------------------------
diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt b/hadoop-common-project/hadoop-common/CHANGES.txt
index 84e7076..098194c 100644
--- a/hadoop-common-project/hadoop-common/CHANGES.txt
+++ b/hadoop-common-project/hadoop-common/CHANGES.txt
@@ -1023,6 +1023,9 @@ Release 2.8.0 - UNRELEASED
HADOOP-12175. FsShell must load SpanReceierHost to support tracing
(Masatake Iwasaki via Colin P. McCabe)
+ HADOOP-10945. 4-digit octal umask permissions throws a parse error (Chang
+ Li via jlowe)
+
Release 2.7.2 - UNRELEASED
INCOMPATIBLE CHANGES
http://git-wip-us.apache.org/repos/asf/hadoop/blob/6f0a3572/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/UmaskParser.java
----------------------------------------------------------------------
diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/UmaskParser.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/UmaskParser.java
index 79956c5..0cf26c5 100644
--- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/UmaskParser.java
+++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/permission/UmaskParser.java
@@ -33,7 +33,7 @@ import org.apache.hadoop.classification.InterfaceStability;
@InterfaceStability.Unstable
class UmaskParser extends PermissionParser {
private static Pattern chmodOctalPattern =
- Pattern.compile("^\\s*[+]?()([0-7]{3})\\s*$"); // no leading 1 for sticky bit
+ Pattern.compile("^\\s*[+]?(0*)([0-7]{3})\\s*$"); // no leading 1 for sticky bit
private static Pattern umaskSymbolicPattern = /* not allow X or t */
Pattern.compile("\\G\\s*([ugoa]*)([+=-]+)([rwx]*)([,\\s]*)\\s*");
final short umaskMode;
http://git-wip-us.apache.org/repos/asf/hadoop/blob/6f0a3572/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/security/TestPermission.java
----------------------------------------------------------------------
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/security/TestPermission.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/security/TestPermission.java
index 036aaae..425c82e 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/security/TestPermission.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/security/TestPermission.java
@@ -20,6 +20,7 @@ package org.apache.hadoop.security;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.fail;
import java.io.IOException;
import java.util.Random;
@@ -100,6 +101,31 @@ public class TestPermission {
conf = new Configuration();
conf.set(FsPermission.UMASK_LABEL, "022");
assertEquals(18, FsPermission.getUMask(conf).toShort());
+
+ // Test 5 - equivalent valid umask
+ conf = new Configuration();
+ conf.set(FsPermission.UMASK_LABEL, "0022");
+ assertEquals(18, FsPermission.getUMask(conf).toShort());
+
+ // Test 6 - invalid umask
+ conf = new Configuration();
+ conf.set(FsPermission.UMASK_LABEL, "1222");
+ try {
+ FsPermission.getUMask(conf);
+ fail("expect IllegalArgumentException happen");
+ } catch (IllegalArgumentException e) {
+ //pass, exception successfully trigger
+ }
+
+ // Test 7 - invalid umask
+ conf = new Configuration();
+ conf.set(FsPermission.UMASK_LABEL, "01222");
+ try {
+ FsPermission.getUMask(conf);
+ fail("expect IllegalArgumentException happen");
+ } catch (IllegalArgumentException e) {
+ //pass, exception successfully trigger
+ }
}
@Test