Need reviewer for 6925932: (file) Path.endsWith can throw ArrayIndexOutOfBoundsException (unx)

Christopher Hegarty - Sun Microsystems Ireland Christopher.Hegarty at Sun.COM
Fri Feb 19 05:30:02 PST 2010


Looks fine.

Trivially, I would retrieve and store path.length and that.path.length 
in locals after checkPath(other), to prevent retrieving their values 
potentially more than once.

-Chris.

On 12/02/2010 16:09, Alan Bateman wrote:
> This is small bug in the Unix implementation of Path.endsWith. Where the
> given path has fewer name elements than the base path but they are
> longer than the corresponding name elements in the base path then the
> comparison can fail with ArrayIndexOutOfBoundException. The reason is
> that the original implementation is missing a length check and is
> trivially fixed. The webrev is here and the test cases expanded to
> better exercise this method:
> http://cr.openjdk.java.net/~alanb/6925932/webrev.00/
>
> Thanks,
> Alan.


More information about the nio-dev mailing list