On 06/25/2013 05:25 AM, Alan Bateman wrote:
On 25/06/2013 01:17, Dan Xu wrote:
HiAll,
The fix of JDK-8013827 added an unnecessary "read" permission
requirement in File.createTempFile methods. This change is going to
fix this issue. Please help review it. Thanks!
webrev: http://cr.openjdk.java.net/~dxu/8017212/webrev.00/
bug: http://bugs.sun.com/view_bug.do?bug_id=8017212
This looks fine. Have you considered adding a test? (as it managed to
slip through with the recent change).
-Alan
Thanks, Alan.
I have updated my patch and added more tests for the permission check.
The new webrev can be reviewed at,
http://cr.openjdk.java.net/~dxu/8017212/webrev.01/
<http://cr.openjdk.java.net/%7Edxu/8017212/webrev.01/>
-Dan
