Mandy Chung wrote:

Alan, Sean,

Can you review the fix for: 6891707: Eliminate the java.io.FilePermission dependency on PolicyFile

Webrev: http://cr.openjdk.java.net/~mchung/6891707/webrev.00/

FilePermission and PolicyFile are the two callers to the sun.security.provider.PolicyFile.canonPath method. Adding a new class for the canonPath method seems unnecessary. So the fix just duplicates the code in FilePermission class.

Thanks Mandy This looks okay to me. Minor comment is this is just a wrapper about getCanonicalPath to avoid calling it with paths that are intended to match all entries in the directory. It might be good to include this in a comment in both places (and this would avoid needing to say that it duplicated in FilePermission).

-Alan