JDK-8224642: Test sun/tools/jcmd/TestJcmdSanity.java fails: Bad file descriptor

[Schmelter, Ralf]

Hi,

> So you prefer the fix in VirtualMachineImpl SocketInputStream close()
> rather than the JCmd try with resources.

Yes, since the close() method is not spec conform. I think it is better to fix it than to adjust the code using it. Especially if you consider that the current behavior could be used to 'steal' a file descriptor.

Best regards,
Ralf