RFR : 8217093: Support extended-length paths in parse_manifest.c on windows

Alan Bateman Alan.Bateman at oracle.com
Thu Jan 24 12:04:22 UTC 2019


On 23/01/2019 08:29, Baesken, Matthias wrote:
> Hello Alan,  here is a second webrev :
>
> http://cr.openjdk.java.net/~mbaesken/webrevs/8217093.1/
>
> I moved the  Windows-only code into a new file
>
> src/java.base/windows/native/libjli/parse_manifest_md.c
>
Did you consider adding a Unix version of open_jarfile to avoid #ifdef 
WIN32 ?

On the Windows version then your patch is low risk but the retrying 
after ENOENT is a bit icky. I'm just wondering if you've considered 
creating an absolute path and adding the prefix when the length is 247 
or greater. I also wonder about the share mode specified to CreateFileW 
where it might be better to specify FILE_SHARE_READ at least so that you 
don't exclude other processes from also opening the file to read.

-Alan


More information about the core-libs-dev mailing list