On 09/10/2018 12:40, Baesken, Matthias wrote:
Hello, please review the following change .
It enhances a number of JNU_ThrowIOExceptionWithLastError  calls with path and 
current working directory information.
For this, a new function JNU_ThrowIOExceptionWithLastErrorAndPath is added.

bug and webrev :

https://bugs.openjdk.java.net/browse/JDK-8211752

http://cr.openjdk.java.net/~mbaesken/webrevs/8211752.0/

This patch will need security review as it leaks sensitive information into exception messages. It may be that it needs to predicated on a security property.

If this proposal goes ahead then the implementation needs cleanup. The upcall from the JNU_* function to jdk.internal.util is a messy I think we need to look at alternative to avoid this. Also the patch needs to be re-formatted to get it consistent with the existing code (e.g. overly long 150 character lines need to be trimmed down to be consistent with the existing code for example).

-Alan

Reply via email to