File: KeepAliveStreamFinalizer.java

package info (click to toggle)
openjdk-21 21.0.8%2B9-1
  • links: PTS, VCS
  • area: main
  • in suites: forky, sid, trixie
  • size: 823,976 kB
  • sloc: java: 5,613,338; xml: 1,643,607; cpp: 1,296,296; ansic: 420,291; asm: 404,850; objc: 20,994; sh: 15,271; javascript: 11,245; python: 6,895; makefile: 2,362; perl: 357; awk: 351; sed: 172; jsp: 24; csh: 3
file content (293 lines) | stat: -rw-r--r-- 10,683 bytes parent folder | download | duplicates (7)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
/*
 * Copyright (c) 2022, Oracle and/or its affiliates. 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
 * @bug 8240275
 * @library /test/lib
 * @run main/othervm KeepAliveStreamFinalizer
 * @summary HttpsURLConnection: connection must not be reused after finalization
 */

import javax.net.ssl.HandshakeCompletedListener;
import javax.net.ssl.HttpsURLConnection;
import javax.net.ssl.SSLSession;
import javax.net.ssl.SSLSocket;
import javax.net.ssl.SSLSocketFactory;
import java.io.IOException;
import java.io.InputStream;
import java.io.OutputStream;
import java.io.OutputStreamWriter;
import java.net.InetAddress;
import java.net.InetSocketAddress;
import java.net.ServerSocket;
import java.net.Socket;
import java.net.URL;
import java.net.UnknownHostException;

public class KeepAliveStreamFinalizer {

    static Server server;
    private static volatile String failureReason;

    static class Server extends Thread {
        final ServerSocket srv;
        static final byte[] requestEnd = new byte[] {'\r', '\n', '\r', '\n'};

        Server(ServerSocket s) {
            srv = s;
        }

        boolean readOneRequest(InputStream is) throws IOException {
            int requestEndCount = 0, r;
            while ((r = is.read()) != -1) {
                if (r == requestEnd[requestEndCount]) {
                    requestEndCount++;
                    if (requestEndCount == 4) {
                        return true;
                    }
                } else {
                    requestEndCount = 0;
                }
            }
            return false;
        }

        public void run() {
            try {
                while (true) {
                    Socket ss = srv.accept();
                    Thread t1 = new Thread(new Runnable() {
                        public void run() {
                            try {
                                InputStream in = ss.getInputStream();
                                OutputStream out = ss.getOutputStream();
                                while (readOneRequest(in)) {
                                    out.write("HTTP/1.1 200 OK\r\nConnection: Keep-Alive\r\nContent-Length: 1\r\n\r\na".getBytes());
                                    out.flush();
                                }
                                in.close();
                                out.close();
                                ss.close();
                            } catch (Exception e) {
                                e.printStackTrace();
                            }
                        }
                    });
                    t1.setDaemon(true);
                    t1.start();
                }
            } catch (Exception e) {
                e.printStackTrace();
            }
        }
    }

    public static void main(String[] args) throws Exception {
        InetSocketAddress address = startHttpServer();
        clientHttpCalls(address);
        if (failureReason != null) {
            throw new RuntimeException(failureReason);
        }
    }

    public static InetSocketAddress startHttpServer() throws Exception {
        InetAddress localHost = InetAddress.getLoopbackAddress();
        InetSocketAddress address = new InetSocketAddress(localHost, 0);
        ServerSocket serverSocket = new ServerSocket();
        serverSocket.bind(address);
        server = new Server(serverSocket);
        server.setDaemon(true);
        server.start();
        return (InetSocketAddress) serverSocket.getLocalSocketAddress();
    }

    public static void doRequest(URL url) throws IOException {
        HttpsURLConnection c = (HttpsURLConnection)url.openConnection();
        c.setRequestMethod("POST");
        c.setDoOutput(true);
        OutputStreamWriter out = new OutputStreamWriter(c.getOutputStream());
        out.write("test");
        out.close();
        int responseCode = c.getResponseCode();
        // Fully reading the body causes the HttpsClient to be added to the KeepAliveCache immediately,
        // which avoids this issue since GC will not finalize the HttpsClient.
    }

    public static void clientHttpCalls(InetSocketAddress address) throws Exception {
        try {
            System.err.println("http server listen on: " + address.getPort());
            String hostAddr = address.getAddress().getHostAddress();
            if (hostAddr.indexOf(':') > -1) hostAddr = "[" + hostAddr + "]";
            String baseURLStr = "https://" + hostAddr + ":" + address.getPort() + "/";

            URL testUrl = new URL(baseURLStr);

            // CheckFinalizeSocketFactory is not a real SSLSocketFactory;
            // it produces regular non-SSL sockets. Effectively, the request
            // is made over http.
            HttpsURLConnection.setDefaultSSLSocketFactory(new CheckFinalizeSocketFactory());
            // now perform up to 3 requests; with the broken KeepAliveStream finalizer,
            // the second request usually attempts to use a finalized socket
            for (int i = 0; i < 3; i++) {
                System.err.println("Request #" + (i + 1));
                doRequest(testUrl);
                System.gc();
                Thread.sleep(100);
                if (failureReason != null) break;
            }
        } finally {
            server.srv.close();
        }
    }

    static class CheckFinalizeSocket extends SSLSocket {
        private volatile boolean finalized;
        public void finalize() throws Throwable {
            System.err.println("In finalize");
            super.finalize();
            finalized = true;
        }

        @Override
        public InputStream getInputStream() throws IOException {
            if (finalized) {
                System.err.println(failureReason = "getInputStream called after finalize");
                Thread.dumpStack();
            }
            return super.getInputStream();
        }

        @Override
        public OutputStream getOutputStream() throws IOException {
            if (finalized) {
                System.err.println(failureReason = "getOutputStream called after finalize");
                Thread.dumpStack();
            }
            return super.getOutputStream();
        }

        @Override
        public synchronized void close() throws IOException {
            if (finalized) {
                System.err.println(failureReason = "close called after finalize");
                Thread.dumpStack();
            }
            super.close();
        }

        // required abstract method overrides
        @Override
        public String[] getSupportedCipherSuites() {
            return new String[0];
        }
        @Override
        public String[] getEnabledCipherSuites() {
            return new String[0];
        }
        @Override
        public void setEnabledCipherSuites(String[] suites) { }
        @Override
        public String[] getSupportedProtocols() {
            return new String[0];
        }
        @Override
        public String[] getEnabledProtocols() {
            return new String[0];
        }
        @Override
        public void setEnabledProtocols(String[] protocols) { }
        @Override
        public SSLSession getSession() {
            return null;
        }
        @Override
        public void addHandshakeCompletedListener(HandshakeCompletedListener listener) { }
        @Override
        public void removeHandshakeCompletedListener(HandshakeCompletedListener listener) { }
        @Override
        public void startHandshake() throws IOException { }
        @Override
        public void setUseClientMode(boolean mode) { }
        @Override
        public boolean getUseClientMode() {
            return false;
        }
        @Override
        public void setNeedClientAuth(boolean need) { }
        @Override
        public boolean getNeedClientAuth() {
            return false;
        }
        @Override
        public void setWantClientAuth(boolean want) { }
        @Override
        public boolean getWantClientAuth() {
            return false;
        }
        @Override
        public void setEnableSessionCreation(boolean flag) { }
        @Override
        public boolean getEnableSessionCreation() {
            return false;
        }
    }

    static class CheckFinalizeSocketFactory extends SSLSocketFactory {

        @Override
        public Socket createSocket() throws IOException {
            return new CheckFinalizeSocket();
        }
        // required abstract method overrides
        @Override
        public Socket createSocket(String host, int port) throws IOException, UnknownHostException {
            throw new UnsupportedOperationException();
        }
        @Override
        public Socket createSocket(String host, int port, InetAddress localHost, int localPort) throws IOException, UnknownHostException {
            throw new UnsupportedOperationException();
        }
        @Override
        public Socket createSocket(InetAddress host, int port) throws IOException {
            throw new UnsupportedOperationException();
        }
        @Override
        public Socket createSocket(InetAddress address, int port, InetAddress localAddress, int localPort) throws IOException {
            throw new UnsupportedOperationException();
        }
        @Override
        public String[] getDefaultCipherSuites() {
            return new String[0];
        }
        @Override
        public String[] getSupportedCipherSuites() {
            return new String[0];
        }
        @Override
        public Socket createSocket(Socket s, String host, int port, boolean autoClose) throws IOException {
            throw new UnsupportedOperationException();
        }
    }
}