From: Dalibor Topic Date: Fri, 29 Apr 2005 18:47:42 +0000 (+0000) Subject: 2005-04-29 Dalibor Topic X-Git-Tag: misc/cutover-cvs2svn~3586 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b61ae8b2611643a3f7005680573e4f13dcf9ef23;p=thirdparty%2Fgcc.git 2005-04-29 Dalibor Topic * java/nio/channels/FileChannelImpl.java (FileChannelImpl(String, int)): Removed. (FileChannelImpl(File, int)): Added. Check if opened file is a directory. * java/io/FileInputStream.java(FileInputStream): Fixed javadocs. Call FileChannelImpl(File, int). * java/io/FileOutputStream.java (FileInputStream): Call FileChannelImpl(File, int). * java/io/RandomAccessFile.java (RandomAccessFile): Call FileChannelImpl(File, int). Switched constructors around. From-SVN: r99011 --- diff --git a/libjava/ChangeLog b/libjava/ChangeLog index 68bee9172f05..5a8022eecf81 100644 --- a/libjava/ChangeLog +++ b/libjava/ChangeLog @@ -1,3 +1,16 @@ +2005-04-29 Dalibor Topic + + * java/nio/channels/FileChannelImpl.java + (FileChannelImpl(String, int)): Removed. + (FileChannelImpl(File, int)): Added. Check if opened file is a + directory. + * java/io/FileInputStream.java(FileInputStream): Fixed javadocs. + Call FileChannelImpl(File, int). + * java/io/FileOutputStream.java (FileInputStream): Call + FileChannelImpl(File, int). + * java/io/RandomAccessFile.java (RandomAccessFile): + Call FileChannelImpl(File, int). Switched constructors around. + 2005-04-27 Andrew Haley PR java/19285 diff --git a/libjava/gnu/java/nio/channels/FileChannelImpl.java b/libjava/gnu/java/nio/channels/FileChannelImpl.java index 9d978c923f71..aaa4c2608647 100644 --- a/libjava/gnu/java/nio/channels/FileChannelImpl.java +++ b/libjava/gnu/java/nio/channels/FileChannelImpl.java @@ -41,6 +41,7 @@ package gnu.java.nio.channels; import gnu.classpath.Configuration; import gnu.java.nio.FileLockImpl; +import java.io.File; import java.io.FileNotFoundException; import java.io.IOException; import java.nio.ByteBuffer; @@ -102,10 +103,27 @@ public final class FileChannelImpl extends FileChannel } /* Open a file. MODE is a combination of the above mode flags. */ - public FileChannelImpl (String path, int mode) throws FileNotFoundException + public FileChannelImpl (File file, int mode) throws FileNotFoundException { + final String path = file.getPath(); fd = open (path, mode); this.mode = mode; + + // First open the file and then check if it is a a directory + // to avoid race condition. + if (file.isDirectory()) + { + try + { + close(); + } + catch (IOException e) + { + /* ignore it */ + } + + throw new FileNotFoundException(path + " is a directory"); + } } /* Used by init() (native code) */ diff --git a/libjava/java/io/FileInputStream.java b/libjava/java/io/FileInputStream.java index ebd5d20ad464..c719955434f6 100644 --- a/libjava/java/io/FileInputStream.java +++ b/libjava/java/io/FileInputStream.java @@ -76,7 +76,8 @@ public class FileInputStream extends InputStream * @param name The name of the file this stream should read from * * @exception SecurityException If read access to the file is not allowed - * @exception FileNotFoundException If the file does not exist. + * @exception FileNotFoundException If the file does not exist + * or if it is a directory */ public FileInputStream(String name) throws FileNotFoundException { @@ -97,7 +98,8 @@ public class FileInputStream extends InputStream * @param file The File object this stream should read from * * @exception SecurityException If read access to the file is not allowed - * @exception FileNotFoundException If the file does not exist. + * @exception FileNotFoundException If the file does not exist + * or if it is a directory. */ public FileInputStream(File file) throws FileNotFoundException { @@ -105,7 +107,7 @@ public class FileInputStream extends InputStream if (s != null) s.checkRead(file.getPath()); - ch = new FileChannelImpl (file.getPath(), FileChannelImpl.READ); + ch = new FileChannelImpl (file, FileChannelImpl.READ); } /** diff --git a/libjava/java/io/FileOutputStream.java b/libjava/java/io/FileOutputStream.java index d5fa2d3bb99c..e8784429e63a 100644 --- a/libjava/java/io/FileOutputStream.java +++ b/libjava/java/io/FileOutputStream.java @@ -155,10 +155,10 @@ public class FileOutputStream extends OutputStream if (s != null) s.checkWrite(file.getPath()); - ch = new FileChannelImpl (file.getPath(), (append - ? FileChannelImpl.WRITE - | FileChannelImpl.APPEND - : FileChannelImpl.WRITE)); + ch = new FileChannelImpl (file, (append + ? FileChannelImpl.WRITE + | FileChannelImpl.APPEND + : FileChannelImpl.WRITE)); } /** diff --git a/libjava/java/io/RandomAccessFile.java b/libjava/java/io/RandomAccessFile.java index c23ca3adf2e8..ef367949a24c 100644 --- a/libjava/java/io/RandomAccessFile.java +++ b/libjava/java/io/RandomAccessFile.java @@ -86,37 +86,11 @@ public class RandomAccessFile implements DataOutput, DataInput * illegal value * @exception SecurityException If the requested access to the file * is not allowed - * @exception IOException If any other error occurs + * @exception FileNotFoundException If the file is a directory, or + * any other error occurs */ public RandomAccessFile (File file, String mode) throws FileNotFoundException - { - this (file.getPath(), mode); - } - - /** - * This method initializes a new instance of RandomAccessFile - * to read from the specified file name with the specified access mode. - * The access mode is either "r" for read only access, "rw" for read - * write access, "rws" for synchronized read/write access of both - * content and metadata, or "rwd" for read/write access - * where only content is required to be synchronous. - *

- * Note that a SecurityManager check is made prior to - * opening the file to determine whether or not this file is allowed to - * be read or written. - * - * @param fileName The name of the file to read and/or write - * @param mode "r", "rw", "rws", or "rwd" - * - * @exception IllegalArgumentException If mode has an - * illegal value - * @exception SecurityException If the requested access to the file - * is not allowed - * @exception FileNotFoundException If any other error occurs - */ - public RandomAccessFile (String fileName, String mode) - throws FileNotFoundException { int fdmode; if (mode.equals("r")) @@ -136,6 +110,8 @@ public class RandomAccessFile implements DataOutput, DataInput else throw new IllegalArgumentException ("invalid mode: " + mode); + final String fileName = file.getPath(); + // The obligatory SecurityManager stuff SecurityManager s = System.getSecurityManager(); if (s != null) @@ -146,12 +122,40 @@ public class RandomAccessFile implements DataOutput, DataInput s.checkWrite(fileName); } - ch = new FileChannelImpl (fileName, fdmode); + ch = new FileChannelImpl (file, fdmode); fd = new FileDescriptor(ch); out = new DataOutputStream (new FileOutputStream (fd)); in = new DataInputStream (new FileInputStream (fd)); } + /** + * This method initializes a new instance of RandomAccessFile + * to read from the specified file name with the specified access mode. + * The access mode is either "r" for read only access, "rw" for read + * write access, "rws" for synchronized read/write access of both + * content and metadata, or "rwd" for read/write access + * where only content is required to be synchronous. + *

+ * Note that a SecurityManager check is made prior to + * opening the file to determine whether or not this file is allowed to + * be read or written. + * + * @param fileName The name of the file to read and/or write + * @param mode "r", "rw", "rws", or "rwd" + * + * @exception IllegalArgumentException If mode has an + * illegal value + * @exception SecurityException If the requested access to the file + * is not allowed + * @exception FileNotFoundException If the file is a directory or + * any other error occurs + */ + public RandomAccessFile (String fileName, String mode) + throws FileNotFoundException + { + this (new File(fileName), mode); + } + /** * This method closes the file and frees up all file related system * resources. Since most operating systems put a limit on how many files