author | rriggs |
Fri, 02 Feb 2018 14:17:07 -0500 | |
changeset 48737 | 7c12219870fd |
parent 48224 | be0df5ab3093 |
permissions | -rw-r--r-- |
2 | 1 |
/* |
48737
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
2 |
* Copyright (c) 2003, 2018, Oracle and/or its affiliates. All rights reserved. |
2 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
5506 | 7 |
* published by the Free Software Foundation. Oracle designates this |
2 | 8 |
* particular file as subject to the "Classpath" exception as provided |
5506 | 9 |
* by Oracle in the LICENSE file that accompanied this code. |
2 | 10 |
* |
11 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
12 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
13 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
14 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
15 |
* accompanied this code). |
|
16 |
* |
|
17 |
* You should have received a copy of the GNU General Public License version |
|
18 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
19 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
20 |
* |
|
5506 | 21 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
22 |
* or visit www.oracle.com if you need additional information or have any |
|
23 |
* questions. |
|
2 | 24 |
*/ |
25 |
||
26 |
package java.io; |
|
27 |
||
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
28 |
import java.lang.ref.Cleaner; |
11017
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
29 |
import java.util.ArrayList; |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
30 |
import java.util.List; |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
31 |
|
32834
e1dca5fe4de3
8137056: Move SharedSecrets and interface friends out of sun.misc
chegar
parents:
28425
diff
changeset
|
32 |
import jdk.internal.misc.JavaIOFileDescriptorAccess; |
e1dca5fe4de3
8137056: Move SharedSecrets and interface friends out of sun.misc
chegar
parents:
28425
diff
changeset
|
33 |
import jdk.internal.misc.SharedSecrets; |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
34 |
import jdk.internal.ref.CleanerFactory; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
35 |
import jdk.internal.ref.PhantomCleanable; |
2 | 36 |
|
37 |
/** |
|
38 |
* Instances of the file descriptor class serve as an opaque handle |
|
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
39 |
* to the underlying machine-specific structure representing an open |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
40 |
* file, an open socket, or another source or sink of bytes. |
48
dc5744ca15ea
4960438: (process) Need IO redirection API for subprocesses
martin
parents:
2
diff
changeset
|
41 |
* The main practical use for a file descriptor is to create a |
dc5744ca15ea
4960438: (process) Need IO redirection API for subprocesses
martin
parents:
2
diff
changeset
|
42 |
* {@link FileInputStream} or {@link FileOutputStream} to contain it. |
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
43 |
* <p> |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
44 |
* Applications should not create their own file descriptors. |
2 | 45 |
* |
46 |
* @author Pavani Diwanji |
|
24865
09b1d992ca72
8044740: Convert all JDK versions used in @since tag to 1.n[.n] in jdk repo
henryjen
parents:
11017
diff
changeset
|
47 |
* @since 1.0 |
2 | 48 |
*/ |
49 |
public final class FileDescriptor { |
|
50 |
||
51 |
private int fd; |
|
52 |
||
53 |
private long handle; |
|
54 |
||
11017
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
55 |
private Closeable parent; |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
56 |
private List<Closeable> otherParents; |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
57 |
private boolean closed; |
2 | 58 |
|
59 |
/** |
|
27263
819f5f87d485
8023173: FileDescriptor should respect append flag
igerasim
parents:
25859
diff
changeset
|
60 |
* true, if file is opened for appending. |
819f5f87d485
8023173: FileDescriptor should respect append flag
igerasim
parents:
25859
diff
changeset
|
61 |
*/ |
819f5f87d485
8023173: FileDescriptor should respect append flag
igerasim
parents:
25859
diff
changeset
|
62 |
private boolean append; |
819f5f87d485
8023173: FileDescriptor should respect append flag
igerasim
parents:
25859
diff
changeset
|
63 |
|
2 | 64 |
static { |
65 |
initIDs(); |
|
66 |
} |
|
67 |
||
68 |
// Set up JavaIOFileDescriptorAccess in SharedSecrets |
|
69 |
static { |
|
32834
e1dca5fe4de3
8137056: Move SharedSecrets and interface friends out of sun.misc
chegar
parents:
28425
diff
changeset
|
70 |
SharedSecrets.setJavaIOFileDescriptorAccess( |
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
71 |
new JavaIOFileDescriptorAccess() { |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
72 |
public void set(FileDescriptor fdo, int fd) { |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
73 |
fdo.fd = fd; |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
74 |
} |
2 | 75 |
|
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
76 |
public int get(FileDescriptor fdo) { |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
77 |
return fdo.fd; |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
78 |
} |
48
dc5744ca15ea
4960438: (process) Need IO redirection API for subprocesses
martin
parents:
2
diff
changeset
|
79 |
|
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
80 |
public void setAppend(FileDescriptor fdo, boolean append) { |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
81 |
fdo.append = append; |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
82 |
} |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
83 |
|
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
84 |
public boolean getAppend(FileDescriptor fdo) { |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
85 |
return fdo.append; |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
86 |
} |
27263
819f5f87d485
8023173: FileDescriptor should respect append flag
igerasim
parents:
25859
diff
changeset
|
87 |
|
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
88 |
public void close(FileDescriptor fdo) throws IOException { |
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
89 |
fdo.close(); |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
90 |
} |
27263
819f5f87d485
8023173: FileDescriptor should respect append flag
igerasim
parents:
25859
diff
changeset
|
91 |
|
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
92 |
public void registerCleanup(FileDescriptor fdo) { |
48737
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
93 |
fdo.registerCleanup(null); |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
94 |
} |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
95 |
|
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
96 |
public void registerCleanup(FileDescriptor fdo, PhantomCleanable<Object> cleanup) { |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
97 |
fdo.registerCleanup(cleanup); |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
98 |
} |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
99 |
|
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
100 |
public void unregisterCleanup(FileDescriptor fdo) { |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
101 |
fdo.unregisterCleanup(); |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
102 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
103 |
|
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
104 |
public void setHandle(FileDescriptor fdo, long handle) { |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
105 |
fdo.setHandle(handle); |
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
106 |
} |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
107 |
|
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
108 |
public long getHandle(FileDescriptor fdo) { |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
109 |
return fdo.handle; |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
110 |
} |
48
dc5744ca15ea
4960438: (process) Need IO redirection API for subprocesses
martin
parents:
2
diff
changeset
|
111 |
} |
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
112 |
); |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
113 |
} |
48
dc5744ca15ea
4960438: (process) Need IO redirection API for subprocesses
martin
parents:
2
diff
changeset
|
114 |
|
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
115 |
/** |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
116 |
* Cleanup in case FileDescriptor is not explicitly closed. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
117 |
*/ |
48737
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
118 |
private PhantomCleanable<Object> cleanup; |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
119 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
120 |
/** |
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
121 |
* Constructs an (invalid) FileDescriptor |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
122 |
* object. |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
123 |
*/ |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
124 |
public FileDescriptor() { |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
125 |
fd = -1; |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
126 |
handle = -1; |
2 | 127 |
} |
128 |
||
129 |
/** |
|
130 |
* A handle to the standard input stream. Usually, this file |
|
131 |
* descriptor is not used directly, but rather via the input stream |
|
48
dc5744ca15ea
4960438: (process) Need IO redirection API for subprocesses
martin
parents:
2
diff
changeset
|
132 |
* known as {@code System.in}. |
2 | 133 |
* |
134 |
* @see java.lang.System#in |
|
135 |
*/ |
|
136 |
public static final FileDescriptor in = standardStream(0); |
|
137 |
||
138 |
/** |
|
139 |
* A handle to the standard output stream. Usually, this file |
|
140 |
* descriptor is not used directly, but rather via the output stream |
|
48
dc5744ca15ea
4960438: (process) Need IO redirection API for subprocesses
martin
parents:
2
diff
changeset
|
141 |
* known as {@code System.out}. |
2 | 142 |
* @see java.lang.System#out |
143 |
*/ |
|
144 |
public static final FileDescriptor out = standardStream(1); |
|
145 |
||
146 |
/** |
|
147 |
* A handle to the standard error stream. Usually, this file |
|
148 |
* descriptor is not used directly, but rather via the output stream |
|
48
dc5744ca15ea
4960438: (process) Need IO redirection API for subprocesses
martin
parents:
2
diff
changeset
|
149 |
* known as {@code System.err}. |
2 | 150 |
* |
151 |
* @see java.lang.System#err |
|
152 |
*/ |
|
153 |
public static final FileDescriptor err = standardStream(2); |
|
154 |
||
155 |
/** |
|
156 |
* Tests if this file descriptor object is valid. |
|
157 |
* |
|
48
dc5744ca15ea
4960438: (process) Need IO redirection API for subprocesses
martin
parents:
2
diff
changeset
|
158 |
* @return {@code true} if the file descriptor object represents a |
2 | 159 |
* valid, open file, socket, or other active I/O connection; |
48
dc5744ca15ea
4960438: (process) Need IO redirection API for subprocesses
martin
parents:
2
diff
changeset
|
160 |
* {@code false} otherwise. |
2 | 161 |
*/ |
162 |
public boolean valid() { |
|
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
163 |
return (handle != -1) || (fd != -1); |
2 | 164 |
} |
165 |
||
166 |
/** |
|
167 |
* Force all system buffers to synchronize with the underlying |
|
168 |
* device. This method returns after all modified data and |
|
169 |
* attributes of this FileDescriptor have been written to the |
|
170 |
* relevant device(s). In particular, if this FileDescriptor |
|
171 |
* refers to a physical storage medium, such as a file in a file |
|
172 |
* system, sync will not return until all in-memory modified copies |
|
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
173 |
* of buffers associated with this FileDescriptor have been |
2 | 174 |
* written to the physical medium. |
175 |
* |
|
176 |
* sync is meant to be used by code that requires physical |
|
177 |
* storage (such as a file) to be in a known state For |
|
178 |
* example, a class that provided a simple transaction facility |
|
179 |
* might use sync to ensure that all changes to a file caused |
|
180 |
* by a given transaction were recorded on a storage medium. |
|
181 |
* |
|
182 |
* sync only affects buffers downstream of this FileDescriptor. If |
|
183 |
* any in-memory buffering is being done by the application (for |
|
184 |
* example, by a BufferedOutputStream object), those buffers must |
|
185 |
* be flushed into the FileDescriptor (for example, by invoking |
|
186 |
* OutputStream.flush) before that data will be affected by sync. |
|
187 |
* |
|
188 |
* @exception SyncFailedException |
|
189 |
* Thrown when the buffers cannot be flushed, |
|
190 |
* or because the system cannot guarantee that all the |
|
191 |
* buffers have been synchronized with physical media. |
|
24865
09b1d992ca72
8044740: Convert all JDK versions used in @since tag to 1.n[.n] in jdk repo
henryjen
parents:
11017
diff
changeset
|
192 |
* @since 1.1 |
2 | 193 |
*/ |
194 |
public native void sync() throws SyncFailedException; |
|
195 |
||
196 |
/* This routine initializes JNI field offsets for the class */ |
|
197 |
private static native void initIDs(); |
|
198 |
||
199 |
private static FileDescriptor standardStream(int fd) { |
|
200 |
FileDescriptor desc = new FileDescriptor(); |
|
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
201 |
desc.handle = getHandle(fd); |
2 | 202 |
return desc; |
203 |
} |
|
204 |
||
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
205 |
private static native long getHandle(int d); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
206 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
207 |
/** |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
208 |
* Set the handle. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
209 |
* If setting to -1, clear the cleaner. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
210 |
* The {@link #registerCleanup()} method should be called for new handles. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
211 |
* @param handle the handle or -1 to indicate closed |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
212 |
*/ |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
213 |
@SuppressWarnings("unchecked") |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
214 |
void setHandle(long handle) { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
215 |
if (handle == -1 && cleanup != null) { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
216 |
cleanup.clear(); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
217 |
cleanup = null; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
218 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
219 |
this.handle = handle; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
220 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
221 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
222 |
/** |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
223 |
* Register a cleanup for the current handle. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
224 |
* Used directly in java.io and indirectly via fdAccess. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
225 |
* The cleanup should be registered after the handle is set in the FileDescriptor. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
226 |
*/ |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
227 |
@SuppressWarnings("unchecked") |
48737
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
228 |
void registerCleanup() { |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
229 |
registerCleanup(null); |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
230 |
} |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
231 |
|
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
232 |
/** |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
233 |
* Register a cleanup for the current handle. |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
234 |
* Used directly in java.io and indirectly via fdAccess. |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
235 |
* The cleanup should be registered after the handle is set in the FileDescriptor. |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
236 |
* @param newCleanable a PhantomCleanable to register |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
237 |
*/ |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
238 |
@SuppressWarnings("unchecked") |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
239 |
synchronized void registerCleanup(PhantomCleanable<Object> newCleanable) { |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
240 |
if (cleanup != null) { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
241 |
cleanup.clear(); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
242 |
} |
48737
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
243 |
cleanup = (newCleanable == null) ? FDCleanup.create(this) : newCleanable; |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
244 |
} |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
245 |
|
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
246 |
/** |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
247 |
* Unregister a cleanup for the current raw fd. |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
248 |
* Used directly in java.io and indirectly via fdAccess. |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
249 |
* Normally {@link #close()} should be used except in cases where |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
250 |
* it is certain the caller will close the raw fd and the cleanup |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
251 |
* must not close the raw fd. {@link #unregisterCleanup()} must be |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
252 |
* called before the raw fd is closed to prevent a race that makes |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
253 |
* it possible for the fd to be reallocated to another use and later |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
254 |
* the cleanup might be invoked. |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
255 |
*/ |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
256 |
synchronized void unregisterCleanup() { |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
257 |
if (cleanup != null) { |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
258 |
cleanup.clear(); |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
259 |
} |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
260 |
cleanup = null; |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
261 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
262 |
|
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
263 |
/** |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
264 |
* Close the raw file descriptor or handle, if it has not already been closed |
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
265 |
* and set the fd and handle to -1. |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
266 |
* Clear the cleaner so the close does not happen twice. |
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
267 |
* Package private to allow it to be used in java.io. |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
268 |
* @throws IOException if close fails |
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
269 |
*/ |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
270 |
@SuppressWarnings("unchecked") |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
271 |
synchronized void close() throws IOException { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
272 |
if (cleanup != null) { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
273 |
cleanup.clear(); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
274 |
cleanup = null; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
275 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
276 |
close0(); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
277 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
278 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
279 |
/* |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
280 |
* Close the raw file descriptor or handle, if it has not already been closed |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
281 |
* and set the fd and handle to -1. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
282 |
*/ |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
283 |
private native void close0() throws IOException; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
284 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
285 |
/* |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
286 |
* Raw close of the file handle. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
287 |
* Used only for last chance cleanup. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
288 |
*/ |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
289 |
private static native void cleanupClose0(long handle) throws IOException; |
47235
9ef10c6e67b8
8187631: Refactor FileDescriptor close implementation
rriggs
parents:
47216
diff
changeset
|
290 |
|
11017
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
291 |
/* |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
292 |
* Package private methods to track referents. |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
293 |
* If multiple streams point to the same FileDescriptor, we cycle |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
294 |
* through the list of all referents and call close() |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
295 |
*/ |
2 | 296 |
|
11017
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
297 |
/** |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
298 |
* Attach a Closeable to this FD for tracking. |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
299 |
* parent reference is added to otherParents when |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
300 |
* needed to make closeAll simpler. |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
301 |
*/ |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
302 |
synchronized void attach(Closeable c) { |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
303 |
if (parent == null) { |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
304 |
// first caller gets to do this |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
305 |
parent = c; |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
306 |
} else if (otherParents == null) { |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
307 |
otherParents = new ArrayList<>(); |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
308 |
otherParents.add(parent); |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
309 |
otherParents.add(c); |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
310 |
} else { |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
311 |
otherParents.add(c); |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
312 |
} |
2 | 313 |
} |
314 |
||
11017
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
315 |
/** |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
316 |
* Cycle through all Closeables sharing this FD and call |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
317 |
* close() on each one. |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
318 |
* |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
319 |
* The caller closeable gets to call close0(). |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
320 |
*/ |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
321 |
@SuppressWarnings("try") |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
322 |
synchronized void closeAll(Closeable releaser) throws IOException { |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
323 |
if (!closed) { |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
324 |
closed = true; |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
325 |
IOException ioe = null; |
28425
241dd1e2d9cb
8068948: Update java.base module to use new try-with-resources statement
darcy
parents:
27263
diff
changeset
|
326 |
try (releaser) { |
11017
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
327 |
if (otherParents != null) { |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
328 |
for (Closeable referent : otherParents) { |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
329 |
try { |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
330 |
referent.close(); |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
331 |
} catch(IOException x) { |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
332 |
if (ioe == null) { |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
333 |
ioe = x; |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
334 |
} else { |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
335 |
ioe.addSuppressed(x); |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
336 |
} |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
337 |
} |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
338 |
} |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
339 |
} |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
340 |
} catch(IOException ex) { |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
341 |
/* |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
342 |
* If releaser close() throws IOException |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
343 |
* add other exceptions as suppressed. |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
344 |
*/ |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
345 |
if (ioe != null) |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
346 |
ex.addSuppressed(ioe); |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
347 |
ioe = ex; |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
348 |
} finally { |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
349 |
if (ioe != null) |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
350 |
throw ioe; |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
351 |
} |
353f81426721
7105952: Improve finalisation for FileInputStream/FileOutputStream/RandomAccessFile
coffeys
parents:
5506
diff
changeset
|
352 |
} |
2 | 353 |
} |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
354 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
355 |
/** |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
356 |
* Cleanup for a FileDescriptor when it becomes phantom reachable. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
357 |
* Create a cleanup if handle != -1. |
48737
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
358 |
* Windows closes files using handles and sockets via the fd. |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
359 |
* Network FileDescriptors using socket fd must provide their |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
360 |
* own PhantomCleanable to {@link #registerCleanup}. |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
361 |
* This implementation only clears thehandles. |
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
362 |
* <p> |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
363 |
* Subclassed from {@code PhantomCleanable} so that {@code clear} can be |
48737
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
364 |
* called to disable the cleanup when the handle is closed by any means other |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
365 |
* than calling {@link FileDescriptor#close}. |
48737
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
366 |
* Otherwise, it may incorrectly close the handle after it has been reused. |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
367 |
*/ |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
368 |
static final class FDCleanup extends PhantomCleanable<Object> { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
369 |
private final long handle; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
370 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
371 |
static FDCleanup create(FileDescriptor fdo) { |
48737
7c12219870fd
8195059: Update java.net Socket and DatagramSocket implementations to use Cleaner
rriggs
parents:
48224
diff
changeset
|
372 |
return fdo.handle == -1L |
48224
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
373 |
? null |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
374 |
: new FDCleanup(fdo, CleanerFactory.cleaner(), fdo.handle); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
375 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
376 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
377 |
/** |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
378 |
* Constructor for a phantom cleanable reference. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
379 |
* @param obj the object to monitor |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
380 |
* @param cleaner the cleaner |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
381 |
* @param handle file handle to close |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
382 |
*/ |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
383 |
private FDCleanup(Object obj, Cleaner cleaner, long handle) { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
384 |
super(obj, cleaner); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
385 |
this.handle = handle; |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
386 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
387 |
|
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
388 |
/** |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
389 |
* Close the native handle. |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
390 |
*/ |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
391 |
@Override |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
392 |
protected void performCleanup() { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
393 |
try { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
394 |
cleanupClose0(handle); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
395 |
} catch (IOException ioe) { |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
396 |
throw new UncheckedIOException("close", ioe); |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
397 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
398 |
} |
be0df5ab3093
8080225: FileInput/OutputStream/FileChannel cleanup should be improved
rriggs
parents:
47235
diff
changeset
|
399 |
} |
2 | 400 |
} |