archive/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2025-08-27 14:54:52 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

8297569: URLPermission constructor throws IllegalArgumentException: Invalid characters in hostname after JDK-8294378

Browse source 
Reviewed-by: dfuchs
This commit is contained in:
Jaikiran Pai 2022-11-25 01:23:53 +00:00
parent 87d1097d9b
commit 2f47f83add
3 changed files with 4 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

3
src/java.base/share/classes/java/net/HostPortrange.java
View file

@ -149,9 +149,6 @@ class HostPortrange {
// regular domain name // regular domain name
hoststr = toLowerCase(hoststr); hoststr = toLowerCase(hoststr);
} }
} else {
// regular domain name
hoststr = toLowerCase(hoststr);
} }
} }
hostname = hoststr; hostname = hoststr;

2
src/java.base/share/classes/java/net/URLPermission.java
View file

@ -494,7 +494,7 @@ public final class URLPermission extends Permission {
auth = authpath.substring(0, delim); auth = authpath.substring(0, delim);
this.path = authpath.substring(delim); this.path = authpath.substring(delim);
} }
this.authority = new Authority(scheme, auth); this.authority = new Authority(scheme, auth.toLowerCase(Locale.ROOT));
} }
private String actions() { private String actions() {

4
test/jdk/java/net/URLPermission/URLPermissionTest.java
View file

@ -402,7 +402,9 @@ public class URLPermissionTest {
static Test[] createTests = { static Test[] createTests = {
createtest("http://user@foo.com/a/b/c"), createtest("http://user@foo.com/a/b/c"),
createtest("http://user:pass@foo.com/a/b/c"), createtest("http://user:pass@foo.com/a/b/c"),
createtest("http://user:@foo.com/a/b/c") createtest("http://user:@foo.com/a/b/c"),
createtest("http://foo_bar"),
createtest("http://foo_bar:12345")
}; };
static boolean failed = false; static boolean failed = false;