File handle leak on Win with Java 11(.0.2)

Signed-off-by: Snjezana Peco <[email protected]>

org.eclipse.lsp4xml/src/main/java/org/eclipse/lsp4xml/commons/ParentProcessWatcher.java

@@ -21,13 +21,16 @@
 import org.eclipse.lsp4j.jsonrpc.MessageConsumer;
 import org.eclipse.lsp4j.services.LanguageServer;
 
+import com.google.common.io.Closeables;
+
 /**
  * Watches the parent process PID and invokes exit if it is no longer available.
  * This implementation waits for periods of inactivity to start querying the PIDs.
  */
 public final class ParentProcessWatcher implements Runnable, Function<MessageConsumer, MessageConsumer>{
 
 	private static final Logger LOGGER = Logger.getLogger(ParentProcessWatcher.class.getName());
+	private static final boolean isJava1x = System.getProperty("java.version").startsWith("1.");
 
 	/**
 	 * Exit code returned when XML Language Server is forced to exit.
@@ -106,6 +109,16 @@ private boolean parentProcessStillRunning() {
 				// On Windows, when the Java LS is idle, we need to explicitly request a GC, 
 				// to prevent an accumulation of zombie processes, as finalize() will be called.
 				if (isWindows) {
+					// Java >= 9 doesn't close the handle when the process is garbage collected
+					// We need to close the opened streams
+					if (!isJava1x) {
+						Closeables.closeQuietly(process.getInputStream());
+						Closeables.closeQuietly(process.getErrorStream());
+						try {
+							Closeables.close(process.getOutputStream(), false);
+						} catch (IOException e) {
+						}
+					}
 					System.gc();
 				}
 			}