author | iignatyev |
Mon, 01 Oct 2018 14:54:46 -0700 | |
changeset 51977 | a8862960c19f |
parent 47216 | 71c04702a3d5 |
child 54314 | 46cf212cdcca |
permissions | -rw-r--r-- |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
1 |
/* |
45285 | 2 |
* Copyright (c) 2009, 2017, Oracle and/or its affiliates. All rights reserved. |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
4 |
* |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
5 |
* This code is free software; you can redistribute it and/or modify it |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
7 |
* published by the Free Software Foundation. |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
8 |
* |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
13 |
* accompanied this code). |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
14 |
* |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
15 |
* You should have received a copy of the GNU General Public License version |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
18 |
* |
5506 | 19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
20 |
* or visit www.oracle.com if you need additional information or have any |
|
21 |
* questions. |
|
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
22 |
*/ |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
23 |
|
42764
5f368ec0cbb4
8170864: java/net/URLClassLoader/closetest/CloseTest.java has undeclared dependencies
skovalev
parents:
38883
diff
changeset
|
24 |
/* |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
25 |
* @test |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
26 |
* @bug 4167874 |
42764
5f368ec0cbb4
8170864: java/net/URLClassLoader/closetest/CloseTest.java has undeclared dependencies
skovalev
parents:
38883
diff
changeset
|
27 |
* @modules java.logging |
5f368ec0cbb4
8170864: java/net/URLClassLoader/closetest/CloseTest.java has undeclared dependencies
skovalev
parents:
38883
diff
changeset
|
28 |
* jdk.httpserver |
45285 | 29 |
* jdk.compiler |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
30 |
* @library ../../../../com/sun/net/httpserver |
45290
51bbbe0b42b4
8181307: tests added/changed by 8166139 should be updated to use the latest testlibrary
iignatyev
parents:
45285
diff
changeset
|
31 |
* /test/lib |
45467
99c87a16a8e4
8181761: add explicit @build actions for jdk.test.lib classes in all :tier2 tests
iignatyev
parents:
45393
diff
changeset
|
32 |
* @build jdk.test.lib.compiler.CompilerUtils |
99c87a16a8e4
8181761: add explicit @build actions for jdk.test.lib classes in all :tier2 tests
iignatyev
parents:
45393
diff
changeset
|
33 |
* jdk.test.lib.util.FileUtils |
51977
a8862960c19f
8211171: move JarUtils to top-level testlibrary
iignatyev
parents:
47216
diff
changeset
|
34 |
* jdk.test.lib.util.JarUtils |
45467
99c87a16a8e4
8181761: add explicit @build actions for jdk.test.lib classes in all :tier2 tests
iignatyev
parents:
45393
diff
changeset
|
35 |
* jdk.test.lib.Platform |
51977
a8862960c19f
8211171: move JarUtils to top-level testlibrary
iignatyev
parents:
47216
diff
changeset
|
36 |
* FileServerHandler |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
37 |
* @run main/othervm CloseTest |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
38 |
* @summary URL-downloaded jar files can consume all available file descriptors |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
39 |
*/ |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
40 |
|
42764
5f368ec0cbb4
8170864: java/net/URLClassLoader/closetest/CloseTest.java has undeclared dependencies
skovalev
parents:
38883
diff
changeset
|
41 |
import java.io.File; |
45285 | 42 |
import java.io.IOException; |
42764
5f368ec0cbb4
8170864: java/net/URLClassLoader/closetest/CloseTest.java has undeclared dependencies
skovalev
parents:
38883
diff
changeset
|
43 |
import java.lang.reflect.Method; |
5f368ec0cbb4
8170864: java/net/URLClassLoader/closetest/CloseTest.java has undeclared dependencies
skovalev
parents:
38883
diff
changeset
|
44 |
import java.net.URLClassLoader; |
5f368ec0cbb4
8170864: java/net/URLClassLoader/closetest/CloseTest.java has undeclared dependencies
skovalev
parents:
38883
diff
changeset
|
45 |
import java.net.InetSocketAddress; |
5f368ec0cbb4
8170864: java/net/URLClassLoader/closetest/CloseTest.java has undeclared dependencies
skovalev
parents:
38883
diff
changeset
|
46 |
import java.net.URL; |
45285 | 47 |
import java.nio.file.Files; |
48 |
import java.nio.file.Path; |
|
49 |
import java.nio.file.Paths; |
|
50 |
||
45393 | 51 |
import jdk.test.lib.compiler.CompilerUtils; |
51977
a8862960c19f
8211171: move JarUtils to top-level testlibrary
iignatyev
parents:
47216
diff
changeset
|
52 |
import jdk.test.lib.util.JarUtils; |
45393 | 53 |
|
42764
5f368ec0cbb4
8170864: java/net/URLClassLoader/closetest/CloseTest.java has undeclared dependencies
skovalev
parents:
38883
diff
changeset
|
54 |
import com.sun.net.httpserver.HttpContext; |
5f368ec0cbb4
8170864: java/net/URLClassLoader/closetest/CloseTest.java has undeclared dependencies
skovalev
parents:
38883
diff
changeset
|
55 |
import com.sun.net.httpserver.HttpServer; |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
56 |
|
45285 | 57 |
import static java.nio.file.StandardCopyOption.REPLACE_EXISTING; |
58 |
||
7983
fb83b663cadc
6829919: URLClassLoader.close() doesn't close resource file if getResourceAsStream(...) was called before
michaelm
parents:
7668
diff
changeset
|
59 |
public class CloseTest extends Common { |
45285 | 60 |
private static final String WORK_DIR = System.getProperty("user.dir") |
61 |
+ "/"; |
|
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
62 |
// |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
63 |
// needs two jar files test1.jar and test2.jar with following structure |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
64 |
// |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
65 |
// com/foo/TestClass |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
66 |
// com/foo/TestClass1 |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
67 |
// com/foo/Resource1 |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
68 |
// com/foo/Resource2 |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
69 |
// |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
70 |
// and a directory hierarchy with the same structure/contents |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
71 |
|
45285 | 72 |
public static void main(String args[]) throws Exception { |
73 |
setup(); |
|
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
74 |
|
45285 | 75 |
startHttpServer(WORK_DIR + "serverRoot/"); |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
76 |
|
45285 | 77 |
String testjar = WORK_DIR + "test.jar"; |
78 |
copyFile(WORK_DIR + "test1.jar", testjar); |
|
79 |
test(testjar, 1); |
|
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
80 |
|
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
81 |
// repeat test with different implementation |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
82 |
// of test.jar (whose TestClass.getValue() returns 2 |
45285 | 83 |
copyFile(WORK_DIR + "test2.jar", testjar); |
84 |
test(testjar, 2); |
|
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
85 |
|
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
86 |
// repeat test using a directory of files |
45285 | 87 |
String testdir = WORK_DIR + "testdir/"; |
88 |
rm_minus_rf(new File(testdir)); |
|
89 |
copyDir(WORK_DIR + "test1/", testdir); |
|
90 |
test(testdir, 1); |
|
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
91 |
|
45285 | 92 |
testdir = WORK_DIR + "testdir/"; |
93 |
rm_minus_rf(new File(testdir)); |
|
94 |
copyDir(WORK_DIR + "test2/", testdir); |
|
95 |
test(testdir, 2); |
|
96 |
getHttpServer().stop(3); |
|
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
97 |
} |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
98 |
|
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
99 |
// create a loader on jarfile (or directory), plus a http loader |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
100 |
// load a class , then look for a resource |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
101 |
// also load a class from http loader |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
102 |
// then close the loader |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
103 |
// check further new classes/resources cannot be loaded |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
104 |
// check jar (or dir) can be deleted |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
105 |
// check existing classes can be loaded |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
106 |
// check boot classes can be loaded |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
107 |
|
45285 | 108 |
static void test(String name, int expectedValue) throws Exception { |
109 |
||
110 |
URL url = new URL("file", null, name); |
|
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
111 |
URL url2 = getServerURL(); |
45285 | 112 |
System.out.println("Doing tests with URL: " + url + " and " + url2); |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
113 |
URL[] urls = new URL[2]; |
45285 | 114 |
urls[0] = url; |
115 |
urls[1] = url2; |
|
116 |
URLClassLoader loader = new URLClassLoader(urls); |
|
117 |
Class testclass = loadClass("com.foo.TestClass", loader, true); |
|
118 |
Class class2 = loadClass("Test", loader, true); // from http |
|
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
119 |
class2.newInstance(); |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
120 |
Object test = testclass.newInstance(); |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
121 |
Method method = testclass.getDeclaredMethods()[0]; // int getValue(); |
45285 | 122 |
int res = (Integer) method.invoke(test); |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
123 |
|
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
124 |
if (res != expectedValue) { |
45285 | 125 |
throw new RuntimeException("wrong value from getValue() [" + res + |
126 |
"/" + expectedValue + "]"); |
|
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
127 |
} |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
128 |
|
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
129 |
// should find /resource1 |
45285 | 130 |
URL u1 = loader.findResource("com/foo/Resource1"); |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
131 |
if (u1 == null) { |
45285 | 132 |
throw new RuntimeException("can't find com/foo/Resource1 in test1.jar"); |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
133 |
} |
45285 | 134 |
loader.close(); |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
135 |
|
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
136 |
// should NOT find /resource2 even though it is in jar |
45285 | 137 |
URL u2 = loader.findResource("com/foo/Resource2"); |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
138 |
if (u2 != null) { |
45285 | 139 |
throw new RuntimeException("com/foo/Resource2 unexpected in test1.jar"); |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
140 |
} |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
141 |
|
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
142 |
// load tests |
45285 | 143 |
loadClass("com.foo.TestClass1", loader, false); |
144 |
loadClass("com.foo.TestClass", loader, true); |
|
145 |
loadClass("java.util.ArrayList", loader, true); |
|
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
146 |
|
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
147 |
// now check we can delete the path |
45285 | 148 |
rm_minus_rf(new File(name)); |
149 |
System.out.println(" ... OK"); |
|
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
150 |
} |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
151 |
|
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
152 |
static HttpServer httpServer; |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
153 |
|
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
154 |
static HttpServer getHttpServer() { |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
155 |
return httpServer; |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
156 |
} |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
157 |
|
45285 | 158 |
static URL getServerURL() throws Exception { |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
159 |
int port = httpServer.getAddress().getPort(); |
45285 | 160 |
String s = "http://127.0.0.1:" + port + "/"; |
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
161 |
return new URL(s); |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
162 |
} |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
163 |
|
45285 | 164 |
static void startHttpServer(String docroot) throws Exception { |
165 |
httpServer = HttpServer.create(new InetSocketAddress(0), 10); |
|
166 |
HttpContext ctx = httpServer.createContext( |
|
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
167 |
"/", new FileServerHandler(docroot) |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
168 |
); |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
169 |
httpServer.start(); |
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
170 |
} |
45285 | 171 |
|
172 |
/** |
|
173 |
* Prepare jars files for the tests |
|
174 |
*/ |
|
175 |
private static void setup () throws IOException { |
|
176 |
String[] tests = new String[]{"test1", "test2"}; |
|
177 |
Path workDir = Paths.get(WORK_DIR); |
|
178 |
Path testSrc = Paths.get(System.getProperty("test.src")); |
|
179 |
for (String test : tests) { |
|
180 |
Path testSrcDir = testSrc.resolve(test); |
|
181 |
Path testTargetDir = workDir.resolve(test); |
|
182 |
// Compile sources for corresponding test |
|
183 |
CompilerUtils.compile(testSrcDir, testTargetDir); |
|
184 |
// Copy all resources |
|
185 |
Path packages = Paths.get("com", "foo"); |
|
186 |
Path copySrcDir = testSrcDir.resolve(packages); |
|
187 |
Path copyTargetDir = testTargetDir.resolve(packages); |
|
188 |
Files.createDirectories(copyTargetDir); |
|
189 |
Path res1 = Paths.get("Resource1"); |
|
190 |
Path res2 = Paths.get("Resource2"); |
|
191 |
Files.copy(copySrcDir.resolve(res1), copyTargetDir.resolve(res1), |
|
192 |
REPLACE_EXISTING); |
|
193 |
Files.copy(copySrcDir.resolve(res2), copyTargetDir.resolve(res2), |
|
194 |
REPLACE_EXISTING); |
|
195 |
// Create jar |
|
196 |
JarUtils.createJarFile(workDir.resolve(test + ".jar"), testTargetDir); |
|
197 |
} |
|
198 |
||
199 |
// Copy and compile server test class |
|
200 |
Path serverRoot = Paths.get("serverRoot"); |
|
201 |
Path targetDir = workDir.resolve(serverRoot); |
|
202 |
Path file = Paths.get("Test.java"); |
|
203 |
Files.createDirectories(targetDir); |
|
204 |
Files.copy(testSrc.resolve(serverRoot).resolve(file), |
|
205 |
targetDir.resolve(file), REPLACE_EXISTING); |
|
206 |
CompilerUtils.compile(targetDir, targetDir); |
|
207 |
} |
|
1940
e81514210873
4167874: URL-downloaded jar files can consume all available file descriptors
michaelm
parents:
diff
changeset
|
208 |
} |