Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
8274883: (se) Selector.open throws IAE when the default file system p…
…rovider is changed to a custom provider

Reviewed-by: alanb, michaelm
  • Loading branch information
mkartashev authored and Michael-Mc-Mahon committed Dec 7, 2021
1 parent 7ea4b19 commit 7217cb7
Show file tree
Hide file tree
Showing 4 changed files with 256 additions and 5 deletions.
4 changes: 2 additions & 2 deletions src/java.base/share/classes/sun/nio/ch/SocketChannelImpl.java
Expand Up @@ -700,11 +700,11 @@ public SocketChannel bind(SocketAddress local) throws IOException {
private SocketAddress unixBind(SocketAddress local) throws IOException {
UnixDomainSockets.checkPermission();
if (local == null) {
return UnixDomainSockets.UNNAMED;
return UnixDomainSockets.unnamed();
} else {
Path path = UnixDomainSockets.checkAddress(local).getPath();
if (path.toString().isEmpty()) {
return UnixDomainSockets.UNNAMED;
return UnixDomainSockets.unnamed();
} else {
// bind to non-empty path
UnixDomainSockets.bind(fd, path);
Expand Down
16 changes: 13 additions & 3 deletions src/java.base/share/classes/sun/nio/ch/UnixDomainSockets.java
Expand Up @@ -44,7 +44,9 @@
class UnixDomainSockets {
private UnixDomainSockets() { }

static final UnixDomainSocketAddress UNNAMED = UnixDomainSocketAddress.of("");
private static class UnnamedHolder {
static final UnixDomainSocketAddress UNNAMED = UnixDomainSocketAddress.of("");
}

private static final boolean supported;

Expand All @@ -71,7 +73,7 @@ static UnixDomainSocketAddress getRevealedLocalAddress(SocketAddress sa) {
// Security check passed
} catch (SecurityException e) {
// Return unnamed address only if security check fails
addr = UNNAMED;
addr = unnamed();
}
return addr;
}
Expand Down Expand Up @@ -133,7 +135,11 @@ static UnixDomainSocketAddress generateTempName() throws IOException {
throw new BindException("Could not locate temporary directory for sockets");
int rnd = random.nextInt(Integer.MAX_VALUE);
try {
Path path = Path.of(dir, "socket_" + rnd);
final Path path = Path.of(dir, "socket_" + rnd);
if (path.getFileSystem().provider() != sun.nio.fs.DefaultFileSystemProvider.instance()) {
throw new UnsupportedOperationException(
"Unix Domain Sockets not supported on non-default file system");
}
return UnixDomainSocketAddress.of(path);
} catch (InvalidPathException e) {
throw new BindException("Invalid temporary directory");
Expand All @@ -160,6 +166,10 @@ static int accept(FileDescriptor fd, FileDescriptor newfd, String[] paths)
return n;
}

static UnixDomainSocketAddress unnamed() {
return UnnamedHolder.UNNAMED;
}

private static native boolean init();

private static native int socket0() throws IOException;
Expand Down
37 changes: 37 additions & 0 deletions test/jdk/java/nio/channels/Selector/CustomFileSystem.java
@@ -0,0 +1,37 @@
/*
* Copyright (c) 2021, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2021, JetBrains s.r.o.. 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
* @summary Verifies that an attempt to call Selector.open() on a non-default
* file system succeeds.
* @build CustomFileSystem CustomFileSystemProvider
* @run main/othervm -Djava.nio.file.spi.DefaultFileSystemProvider=CustomFileSystemProvider CustomFileSystem
*/

public class CustomFileSystem {
public static void main(String args[]) throws java.io.IOException {
java.nio.channels.Selector.open();
}
}
204 changes: 204 additions & 0 deletions test/jdk/java/nio/channels/Selector/CustomFileSystemProvider.java
@@ -0,0 +1,204 @@
/*
* Copyright (c) 2021, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2021, JetBrains s.r.o.. 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.
*/

import java.io.File;
import java.io.IOException;
import java.net.URI;
import java.nio.channels.FileChannel;
import java.nio.channels.SeekableByteChannel;
import java.nio.file.AccessMode;
import java.nio.file.CopyOption;
import java.nio.file.DirectoryStream;
import java.nio.file.FileStore;
import java.nio.file.FileSystem;
import java.nio.file.LinkOption;
import java.nio.file.OpenOption;
import java.nio.file.Path;
import java.nio.file.ReadOnlyFileSystemException;
import java.nio.file.attribute.BasicFileAttributes;
import java.nio.file.attribute.FileAttribute;
import java.nio.file.attribute.FileAttributeView;
import java.nio.file.attribute.UserPrincipalLookupService;
import java.nio.file.spi.FileSystemProvider;
import java.util.Collections;
import java.util.Iterator;
import java.util.Map;
import java.util.Set;

public class CustomFileSystemProvider extends FileSystemProvider {

private final FileSystemProvider defaultProvider;

public CustomFileSystemProvider(FileSystemProvider defaultProvider) {
this.defaultProvider = defaultProvider;
}

FileSystemProvider defaultProvider() {
return defaultProvider;
}

@Override
public String getScheme() {
return "file";
}

@Override
public FileSystem newFileSystem(URI uri, Map<String,?> env) throws IOException {
return defaultProvider.newFileSystem(uri, env);
}

@Override
public FileSystem getFileSystem(URI uri) {
return defaultProvider.getFileSystem(uri);
}

@Override
public Path getPath(URI uri) {
return defaultProvider.getPath(uri);
}

@Override
public void setAttribute(Path file, String attribute, Object value,
LinkOption... options)
throws IOException
{
throw new RuntimeException("not implemented");
}

@Override
public Map<String,Object> readAttributes(Path file, String attributes,
LinkOption... options)
throws IOException
{
throw new RuntimeException("not implemented");
}

@Override
public <A extends BasicFileAttributes> A readAttributes(Path file,
Class<A> type,
LinkOption... options)
throws IOException
{
throw new RuntimeException("not implemented");
}

@Override
public <V extends FileAttributeView> V getFileAttributeView(Path file,
Class<V> type,
LinkOption... options)
{
throw new RuntimeException("not implemented");
}

@Override
public boolean isHidden(Path file) throws IOException {
throw new ReadOnlyFileSystemException();
}

@Override
public boolean isSameFile(Path file, Path other) throws IOException {
throw new RuntimeException("not implemented");
}

@Override
public void checkAccess(Path file, AccessMode... modes)
throws IOException
{
throw new RuntimeException("not implemented");
}

@Override
public void copy(Path source, Path target, CopyOption... options)
throws IOException
{
throw new RuntimeException("not implemented");
}

@Override
public void move(Path source, Path target, CopyOption... options)
throws IOException
{
throw new RuntimeException("not implemented");
}

@Override
public void delete(Path file) throws IOException {
throw new RuntimeException("not implemented");
}

@Override
public void createSymbolicLink(Path link, Path target, FileAttribute<?>... attrs)
throws IOException
{
throw new RuntimeException("not implemented");
}

@Override
public void createLink(Path link, Path existing) throws IOException {
throw new RuntimeException("not implemented");
}

@Override
public Path readSymbolicLink(Path link) throws IOException {
throw new RuntimeException("not implemented");
}

@Override
public void createDirectory(Path dir, FileAttribute<?>... attrs)
throws IOException
{
throw new RuntimeException("not implemented");
}

@Override
public DirectoryStream<Path> newDirectoryStream(Path dir,
DirectoryStream.Filter<? super Path> filter)
throws IOException
{
throw new RuntimeException("not implemented");
}

@Override
public SeekableByteChannel newByteChannel(Path file,
Set<? extends OpenOption> options,
FileAttribute<?>... attrs)
throws IOException
{
throw new RuntimeException("not implemented");
}

@Override
public FileChannel newFileChannel(Path file,
Set<? extends OpenOption> options,
FileAttribute<?>... attrs)
throws IOException
{
throw new RuntimeException("not implemented");
}

@Override
public FileStore getFileStore(Path file) throws IOException {
throw new RuntimeException("not implemented");
}
}

1 comment on commit 7217cb7

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.