6630639: lightweight HttpServer leaks file descriptors on no-data connections

Not cleaning up no-data connections properly

Reviewed-by: chegar
This commit is contained in:
Michael McMahon 2009-03-10 03:18:22 -07:00
parent 6c11535cdd
commit a0ec52da33
2 changed files with 6 additions and 0 deletions

View File

@ -52,6 +52,9 @@ class Request {
os = rawout;
do {
startLine = readLine();
if (startLine == null) {
return;
}
/* skip blank lines */
} while (startLine.equals (""));
}

View File

@ -441,6 +441,7 @@ class ServerImpl implements TimeSource {
rawin = sslStreams.getInputStream();
rawout = sslStreams.getOutputStream();
engine = sslStreams.getSSLEngine();
connection.sslStreams = sslStreams;
} else {
rawin = new BufferedInputStream(
new Request.ReadStream (
@ -450,6 +451,8 @@ class ServerImpl implements TimeSource {
ServerImpl.this, chan
);
}
connection.raw = rawin;
connection.rawout = rawout;
}
Request req = new Request (rawin, rawout);
requestLine = req.requestLine();