Apply FilePermCompat.newPermPlusAltPath to limited privileged permission #3186
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Apply
FilePermCompat.newPermPlusAltPath
to limited privileged permissionApply
FilePermCompat.newPermPlusAltPath
to limited privileged permission before checking incoming permission to ensureFilePermission
compatibility after JDK-8164705.Verified manually that the test suite
JTReg test: sun/security/util/FilePermCompat/CompatImpact.java
now passes. (#3104 fixed a specific sub-test within this test suite)For what it's worth, following is a standalone testcase after digging deep into the regression test suite:
Test code
main
with default permission (i.e., without required security permission)Code with
java.security.AllPermission
Not adding this to the automated test since it is already included in the regression test suite.
Reviewer: @pshipton
FYI: @DanHeidinga
Signed-off-by: Jason Feng fengj@ca.ibm.com