Request for review: 7190897 (fs) Files.isWritable method returns false when the path is writable (win). - ver. 1

Alan Bateman Alan.Bateman at
Fri Mar 1 15:02:10 UTC 2013

On 01/03/2013 11:45, Alexey Utkin wrote:
> That is possible, but includes pre-requirements for installed MS 
> tools. That is not a good idea.
I agree.

> New version of the fix was prepared.
> Bug description:
> The suggested fix:
This looks much better, thank you.

A minor point is that in checkAccessMask then you could call 
OpenThreadToken before the try/finally. That way you wouldn't need to 
check hToken. Otherwise I think this is good to go and it's nice to 
finally fix this issue.


More information about the core-libs-dev mailing list