1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
|
/*
* Copyright (c) 2016, 2025, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License version 2 only, as
* published by the Free Software Foundation.
*
* This code is distributed in the hope that it will be useful, but WITHOUT
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
* version 2 for more details (a copy is included in the LICENSE file that
* accompanied this code).
*
* You should have received a copy of the GNU General Public License version
* 2 along with this work; if not, write to the Free Software Foundation,
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
*
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
* or visit www.oracle.com if you need additional information or have any
* questions.
*/
/**
*
* @test
* @bug 8168127 8354053
* @summary FilePermissionCollection merges incorrectly
* @modules java.base/java.io:open
* @library /test/lib
* @build jdk.test.lib.Asserts
* @run main FilePermissionCollectionMerge
*/
import java.io.FilePermission;
import java.lang.reflect.Method;
import java.security.Permissions;
import jdk.test.lib.Asserts;
@SuppressWarnings("removal")
public class FilePermissionCollectionMerge {
public static void main(String[] args) throws Exception {
test("x");
test("x/*");
test("x/-");
test("*");
test("-");
test("/x");
test("/x/*");
test("/x/-");
}
static void test(String arg) throws Exception {
Method altPathMethod;
Method plusAltPathMethod;
try {
altPathMethod = FilePermission.class.getDeclaredMethod("newPermUsingAltPath");
altPathMethod.setAccessible(true);
plusAltPathMethod = FilePermission.class.getDeclaredMethod("newPermPlusAltPath");
plusAltPathMethod.setAccessible(true);
} catch (Exception ex) {
System.err.println("File permission compatibility initialization failed");
throw ex;
}
FilePermission fp1 = new FilePermission(arg, "read");
FilePermission fp2 = (FilePermission) altPathMethod.invoke(fp1);
FilePermission fp3 = (FilePermission) plusAltPathMethod.invoke(fp1);
// All 3 are different
Asserts.assertNE(fp1, fp2);
Asserts.assertNE(fp1.hashCode(), fp2.hashCode());
Asserts.assertNE(fp1, fp3);
Asserts.assertNE(fp1.hashCode(), fp3.hashCode());
Asserts.assertNE(fp2, fp3);
Asserts.assertNE(fp2.hashCode(), fp3.hashCode());
// The plus one implies the other 2
Asserts.assertTrue(fp3.implies(fp1));
Asserts.assertTrue(fp3.implies(fp2));
// The using one different from original
Asserts.assertFalse(fp2.implies(fp1));
Asserts.assertFalse(fp1.implies(fp2));
// FilePermssionCollection::implies always works
testMerge(fp1);
testMerge(fp2);
testMerge(fp3);
testMerge(fp1, fp2);
testMerge(fp1, fp3);
testMerge(fp2, fp1);
testMerge(fp2, fp3);
testMerge(fp3, fp1);
testMerge(fp3, fp2);
testMerge(fp1, fp2, fp3);
testMerge(fp2, fp3, fp1);
testMerge(fp3, fp1, fp2);
}
// Add all into a collection, and check if it implies the last one.
static void testMerge(FilePermission... fps) {
java.security.Permissions perms = new Permissions();
FilePermission last = null;
for (FilePermission fp : fps) {
perms.add(fp);
last = fp;
}
Asserts.assertTrue(perms.implies(last));
}
}
|
