RFR: 8211752: JNU_ThrowIOExceptionWithLastErrorAndPath - enhance some IOExceptions with path causing the issue
Alan Bateman
Alan.Bateman at oracle.com
Wed Dec 12 10:09:31 UTC 2018
On 11/12/2018 20:00, Sean Mullan wrote:
>
> These exceptions are generated from a very low level part of the
> native JDK Windows or Unix FileSystem implementations. That is a
> concern. The previous usages of this property were more focused and
> confined to smaller parts of the code resulting in fewer code paths to
> analyze.
>
> I think we need to take a step back. I think we need to reconsider
> whether the jdk.includeInExceptions security property is appropriate
> for this type of enhancement.
>
> Therefore, I oppose this change as-is. I'm happy to participate in a
> more involved discussion where we start with use cases and motivation
> before jumping to solutions.
I also dislike this patch and you are probably right that
jdk.includeInExceptions needs to be re-examined or pulled out
completely. In addition to the surprise leakage issue, this patch is
inconsistent as it hanges the exception for only a subset of the file
system operations.
-Alan
More information about the core-libs-dev
mailing list