Repository: hadoop Updated Branches: refs/heads/trunk 2b2bd9214 -> 6f0a35724
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/trunk Commit: 6f0a35724f0da80146dbae4b6f6c341e1d3101f5 Parents: 2b2bd92 Author: Jason Lowe <[email protected]> Authored: Wed Jul 29 15:25:10 2015 +0000 Committer: Jason Lowe <[email protected]> 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
