RFR: 8057558 VirtualMachineImpl.execute on windows should close PipedInputStream before throwing exceptions
Dmitry Samersoff
dmitry.samersoff at oracle.com
Mon Sep 8 20:23:31 UTC 2014
Staffan,
Technically, the fix look OK for me.
But,
executeJCmd("ManagementAgent.start_local").close();
looks a bit weird for me.
It might be better to create something like
executeJCmd_no_out();
-Dmitry
On 2014-09-08 16:25, Staffan Larsen wrote:
> Two fixes:
> - The PipedInputStream used by the attach code on windows is not closed in case of errors
> - The InputStreams returned by VirtualMachine.execute are not closed by all callers
>
> webrev: http://cr.openjdk.java.net/~sla/8057558/webrev.00/
> bug: https://bugs.openjdk.java.net/browse/JDK-8057558
>
> Thanks,
> /Staffan
>
--
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the sources.
More information about the serviceability-dev
mailing list